651676 - Fix MIME type for SVG files on all web servers

Status: RESOLVED FIXED

(mozilla.org Graveyard :: Server Operations, task, P5)

Description

[Fred Wenzel [:wenzel]]

All our web servers (and I am boldly claiming this based on a sample size of 2[1]) serve .svg files with the wrong[2] text/xml MIME type because our Apache versions stem from a century when SVG did not exist yet (j/k).

Anyway, please add this (or something like this) to the respective puppet base classes:

AddType image/svg+xml .svg .svgz
AddEncoding x-gzip .svgz

so that svg shall henceforth be served with the right MIME types by our web servers and not cause bugs like bug 651646 in the future.

CCing various people for their info and as a chance to drop their USD 1/50.

[1] for example: http://people.mozilla.com/~fwenzel/tmp/1104/globe.svg
[2] http://www.mozilla.org/projects/svg/faq.html#source

Comment 1

[Daniel Holbert [:dholbert]]

FWIW, this is already fixed on hg.mozilla.org (in bug 609719 comment 4)

Comment 2

[Daniel Holbert [:dholbert]]

See also the "# SVG" chunk of:
  https://developer.mozilla.org/en/sample_.htaccess_file

Comment 3

[Jake Maul [:jakem]]

This has been added to the puppet webapp::base module configs, for RHEL5 and 6. It will take a while for this to be propagated out to everything, because our systems don't automatically pull new Apache configs and reload them. They'll get worked in naturally over time. This should affect at least everything on the generic and engagement clusters.

Not all web apps (and web severs) are using the webapp::base module at present, so this is not a complete fix.

Comment 4

[Fred Wenzel [:wenzel]]

Sounds good to me! If you want (and have checked it works somewhere), you can close this bug. I am not concerned that this won't show up everywhere immediately, I just wanted this to be in the base module, so future apps don't get deployed with this bug anymore.

Comment 5

[Jake Maul [:jakem]]

I will keep it open at present, but with a reduced priority. I would like to go through the 20+ other httpd.conf files in puppet and work on eliminating some of them, or at least unifying them a bit. This would be part of that.

Comment 6

[Jake Maul [:jakem]]

Changing direction on this. Will open a separate bug to track the unification of Puppet httpd.conf files... the purpose of this bug has been accomplished already.