Correct Content-Type header values for archive downloads.
The content type for both .tar.gz and .tar.bz2 downloads was
application/x-tar, which is correct for .tar files when no
Content-Encoding is present, but is not correct for .tar.gz and .tar.bz2
files unless Content-Encoding is set to gzip or x-bzip2, respectively.
However, setting Content-Encoding causes browsers to undo that encoding
during download, when a .gz or .bz2 file is usually the desired
artifact. Omitting the Content-Encoding header is preferred to avoid
having browsers uncompress non-render-able files.
Additionally, the Content-Disposition line indicates a final desired
filename with .tar.gz or .tar.bz2 extension which makes providing a
Content-Encoding header inappropriate.
With the current configuration browsers (Chrome and Firefox thus far)
are registering the application/x-tar Content-Type and not .tar
extension and appending that extension, yielding filename.tar.gz.tar as
a final on-disk artifact. This was originally reported here:
http://stackoverflow.com/questions/3753659
I've changed the .tar.gz and .tar.bz2 Content-Type values to
application/x-gzip and application/x-bzip2, respectively. Which yields
correctly named download artifacts on Firefox, Chrome, and IE.
#!/bin/sh
cat > writepatterns.py <<EOF
import sys
path = sys.argv[1]
patterns = sys.argv[2:]
fp = file(path, 'wb')
for pattern in patterns:
count = int(pattern[0:-1])
char = pattern[-1] + '\n'
fp.write(char*count)
fp.close()
EOF
echo % prepare repo
hg init a
cd a
# These initial lines of Xs were not in the original file used to generate
# the patch. So all the patch hunks need to be applied to a constant offset
# within this file. If the offset isn't tracked then the hunks can be
# applied to the wrong lines of this file.
python ../writepatterns.py a 34X 10A 1B 10A 1C 10A 1B 10A 1D 10A 1B 10A 1E 10A 1B 10A
hg commit -Am adda
# This is a cleaner patch generated via diff
# In this case it reproduces the problem when
# the output of hg export does not
echo % import patch
hg import -v -m 'b' -d '2 0' - <<EOF
--- a/a 2009-12-08 19:26:17.000000000 -0800
+++ b/a 2009-12-08 19:26:17.000000000 -0800
@@ -9,7 +9,7 @@
A
A
B
-A
+a
A
A
A
@@ -53,7 +53,7 @@
A
A
B
-A
+a
A
A
A
@@ -75,7 +75,7 @@
A
A
B
-A
+a
A
A
A
EOF
echo % compare imported changes against reference file
python ../writepatterns.py aref 34X 10A 1B 1a 9A 1C 10A 1B 10A 1D 10A 1B 1a 9A 1E 10A 1B 1a 9A
diff aref a