incoming mail processor needs to be less case sensitive
Bug #48464 reported by
Stuart Bishop
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Launchpad itself |
Fix Released
|
High
|
Gavin Panella | ||
Bug Description
Got a bounced emailed with the following contents:
The following email was unhandled:
https:/
Error message:
No handler registered for '<email address hidden>'
I suspect that the inbound email processing script is not comparing domain
names case insensitively like it should be.
OOPS-818CEMAIL1
Recently: OOPS-978CEMAIL1, OOPS-1000CEMAIL1, OOPS-1000CEMAIL2, OOPS-1000CEMAIL3
Related branches
lp:~allenap/launchpad/bug-mail-case-sensitive-domain-bug-48464-refactor
- Gavin Panella (community): Approve
-
Diff: 1570 lines (+713/-662)17 files modifiedlib/canonical/launchpad/ftests/test_system_documentation.py (+1/-6)
lib/canonical/launchpad/mail/tests/test_helpers.py (+1/-1)
lib/lp/answers/mail/__init__.py (+5/-0)
lib/lp/answers/mail/handler.py (+95/-0)
lib/lp/answers/tests/emailinterface.txt (+4/-4)
lib/lp/blueprints/doc/spec-mail-exploder.txt (+1/-1)
lib/lp/blueprints/mail/__init__.py (+5/-0)
lib/lp/blueprints/mail/handler.py (+108/-0)
lib/lp/bugs/mail/handler.py (+313/-0)
lib/lp/bugs/mail/tests/test_handler.py (+169/-0)
lib/lp/bugs/tests/bugs-emailinterface.txt (+2/-2)
lib/lp/code/mail/tests/test_codehandler.py (+1/-1)
lib/lp/registry/doc/message-holds.txt (+1/-1)
lib/lp/services/mail/handlers.py (+3/-475)
lib/lp/services/mail/incoming.py (+2/-4)
lib/lp/services/mail/tests/incomingmail.txt (+1/-1)
lib/lp/services/mail/tests/test_handlers.py (+1/-166)
lp:~allenap/launchpad/bug-mail-case-sensitive-domain-bug-48464-fix
- Aaron Bentley (community): Approve
-
Diff: 163 lines (+65/-44)3 files modifiedlib/lp/services/mail/handlers.py (+19/-35)
lib/lp/services/mail/tests/incomingmail.txt (+5/-3)
lib/lp/services/mail/tests/test_handlers.py (+41/-6)
| description: | updated |
| Changed in launchpad: | |
| status: | Unconfirmed → Confirmed |
| description: | updated |
Revision history for this message
| Curtis Hovey (sinzui) wrote | #1 |
Revision history for this message
| Curtis Hovey (sinzui) wrote | #2 |
Revision history for this message
| Curtis Hovey (sinzui) wrote | #3 |
| Changed in launchpad: | |
| assignee: | nobody → gmb |
Revision history for this message
| Ursula Junque (ursinha) wrote | #4 |
Revision history for this message
| Ursula Junque (ursinha) wrote | #5 |
Revision history for this message
| Ursula Junque (ursinha) wrote | #6 |
Revision history for this message
| Ursula Junque (ursinha) wrote | #7 |
Revision history for this message
| Ursula Junque (ursinha) wrote | #8 |
| description: | updated |
| description: | updated |
Revision history for this message
| Robert Collins (lifeless) wrote | #9 |
| Changed in launchpad-foundations: | |
| importance: | Medium → High |
| visibility: | private → public |
Revision history for this message
| Graham Binns (gmb) wrote | #10 |
| Changed in launchpad-foundations: | |
| assignee: | Graham Binns (gmb) → nobody |
| Changed in launchpad-foundations: | |
| assignee: | nobody → Gavin Panella (allenap) |
| status: | Triaged → In Progress |
| tags: | added: bugjam2010 |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote Bug fixed by a commit | #11 |
| tags: | added: qa-needstesting |
| Changed in launchpad-foundations: | |
| status: | In Progress → Fix Committed |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #12 |
| tags: |
added: qa-untestable removed: qa-needstesting |
| Changed in launchpad: | |
| status: | Fix Committed → Fix Released |
| Changed in launchpad: | |
| milestone: | none → 11.01 |
To post a comment you must log in.
We have a similar oops (OOPS-917CEMAIL5) for another Japanese encoded email (iso-2022-jp) send from MS Outlook Exxpress.