Etag in HEAD or GET of a segmented file (manifest) does not work as documented

The documentation says:

The response’s ETag for a GET or HEAD on the manifest file will be the MD5 sum of the concatenated string of ETags for each of the segments in the <container>/<prefix> listing, dynamically.

However, that has not been the case in my tests. I found that quotes were being inserted between the etag strings before concatenating. The following patch should fix this.

=== modified file 'swift/proxy/server.py'
--- swift/proxy/server.py 2011-08-12 13:33:42 +0000
+++ swift/proxy/server.py 2011-10-13 00:30:41 +0000
@@ -854,7 +854,7 @@
                     last_modified = datetime(*map(int, re.split('[^\d]',
                         last_modified)[:-1]))
                     etag = md5(
- '"'.join(o['hash'] for o in listing)).hexdigest()
+ ''.join(o['hash'] for o in listing)).hexdigest()
                 else:
                     content_length = 0
                     last_modified = resp.last_modified