content type fail parsing merge directive

Bug #530455 reported by Tim Penhey
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Launchpad itself
Fix Released
High
Aaron Bentley

Bug Description

lib/lp/code/mail/codehandler.py looks for:
   if part['Content-type'].startswith('text/plain'):

However with a merge directive sent with KMail:

(pdb) part['content-type']
'Text/Plain;\n charset="us-ascii"'

Note the case sensitivity.

Related branches

Revision history for this message
Tim Penhey (thumper) wrote :

http://www.w3.org/Protocols/rfc1341/4_Content-Type.html suggests that the value of the content type should be case-insensitive.

Aaron Bentley (abentley)
Changed in launchpad-code:
status: Triaged → In Progress
assignee: nobody → Aaron Bentley (abentley)
Revision history for this message
Ursula Junque (ursinha) wrote : Bug fixed by a commit
Changed in launchpad-code:
milestone: none → 10.03
status: In Progress → Fix Committed
tags: added: qa-needstesting
Aaron Bentley (abentley)
tags: added: qa-ok
removed: qa-needstesting
Tim Penhey (thumper)
Changed in launchpad-code:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.