Fix traceback on .../msgN/ url...

...this requests the file content and for apache mod_wsgi produced a
traceback because the mime type is None for messages, fixes
issue2550586, thanks to ThomasAH for reporting and to Intevation for
funding the fix.

Author: Ralf Schlatterbeck

CHANGES.txt

@@ -21,6 +21,10 @@ Fixes:
   sent via email. We now check that user has permission on the message
   content and files properties. Thanks to Intevation for funding this
   fix.
+- Fix traceback on .../msgN/ url, this requests the file content and for
+  apache mod_wsgi produced a traceback because the mime type is None for
+  messages, fixes issue2550586, thanks to ThomasAH for reporting and to
+  Intevation for funding the fix.
 
 
 2009-10-09 1.4.10 (r4374)

roundup/cgi/client.py

@@ -880,6 +880,9 @@ def serve_file(self, designator, dre=re.compile(r'([^\d]+)(\d+)')):
                 "this file.")
 
         mime_type = klass.get(nodeid, 'type')
+        # Can happen for msg class:
+        if not mime_type:
+            mime_type = 'text/plain'
 
         # if the mime_type is HTML-ish then make sure we're allowed to serve up
         # HTML-ish content