webapp/error.py uses self.request.response.setStatus() in __init__ and that's bad

Bug #60211 reported by Matthew Paul Thomas
2
Affects Status Importance Assigned to Milestone
Launchpad itself
Triaged
Low
Unassigned

Bug Description

<mpt> oh, I can [use self.request.response.setStatus()] just in __init__
<SteveA> don't do anything in __init__
<mpt> that's what error.py does
<SteveA> then it is wron[g]

Revision history for this message
Steve Alexander (stevea) wrote :

They should be LaunchpadView classes instead, and set the status in initialize().

Changed in launchpad:
importance: Untriaged → Low
Changed in launchpad:
status: Unconfirmed → Confirmed
Curtis Hovey (sinzui)
visibility: private → public
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.