checkwatches should raise a BugTrackerConnectError when Mantis doesn't set a return path

Bug #227334 reported by Graham Binns
2
Affects Status Importance Assigned to Milestone
Launchpad itself
Fix Released
Critical
Gavin Panella

Bug Description

At the moment, if Mantis doesn't set a return path when checkwatches logs in, MantisLoginHandler.rewrite_url() will fail with a BugWatchUpdateWarning (see OOPS-854CCW99).

This is incorrect. This problem is an unrecoverable error, so MantisLoginHandler.rewrite_url() should fail with a BugTrackerConnectError(), which is more meaningful and apt.

Related branches

Changed in malone:
status: New → Confirmed
Gavin Panella (allenap)
tags: added: story-reliable-bug-syncing
Curtis Hovey (sinzui)
tags: added: trivial
Changed in launchpad:
status: Confirmed → Triaged
importance: Undecided → Low
Changed in launchpad:
importance: Low → Critical
Gavin Panella (allenap)
Changed in launchpad:
status: Triaged → In Progress
assignee: nobody → Gavin Panella (allenap)
Revision history for this message
Launchpad QA Bot (lpqabot) wrote :
Changed in launchpad:
milestone: none → 11.02
tags: added: qa-needstesting
Changed in launchpad:
status: In Progress → Fix Committed
Revision history for this message
Robert Collins (lifeless) wrote :

marking untestable after chatting with deryck

tags: added: qa-untestable
removed: qa-needstesting
Revision history for this message
Gavin Panella (allenap) wrote :

Sorry, I should have landed this with --no-qa. Thanks for sorting it out.

William Grant (wgrant)
Changed in launchpad:
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.