Launchpad itself

Launchpad should return a 404 instead of ForbiddenAttribute OOPS

Bug #403618 reported by Ursula Junque
12
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Launchpad itself
Fix Released
High
Gary Poster
Launchpad itself 10.02

Bug Description

As seen on OOPS-1300H667, when trying to access the non-existent URL https://launchpad.net/%7Epkgcrosswire/+archive/ppa/+files/crosswire-ppa_1.6%7E09.04_all.deb/dists, we got an OOPS instead of a graceful 404:

  ForbiddenAttribute: ('publishTraverse', <canonical.launchpad.browser.librarian.StreamOrRedirectLibraryFileAliasView object at 0xc7af890>)

Related branches

lp:~gary/launchpad/bug403618
Guilherme Salgado (community): Approve
Diff: 13 lines (+2/-1)
1 file modified
lib/canonical/launchpad/zcml/librarian.zcml (+2/-1)
Deryck Hodge (deryck)
Changed in launchpad-foundations:
status: New → Triaged
importance: Undecided → High
milestone: none → 3.1.12
Diogo Matsubara (matsubara)
summary: - Launchpad should return a 404 instead of ForbiddenAttibute OOPS
+ Launchpad should return a 404 instead of ForbiddenAttribute OOPS
Diogo Matsubara (matsubara)
Changed in launchpad-foundations:
milestone: 3.1.12 → 10.01
Revision history for this message
Gary Poster (gary) wrote :

Change needed, without test:

=== modified file 'lib/canonical/launchpad/zcml/librarian.zcml'
--- lib/canonical/launchpad/zcml/librarian.zcml 2009-07-13 18:15:02 +0000
+++ lib/canonical/launchpad/zcml/librarian.zcml 2010-02-16 16:09:15 +0000
@@ -44,7 +44,8 @@

   <!-- StreamOrRedirectLibraryFileAliasView -->
   <class class="canonical.launchpad.browser.librarian.StreamOrRedirectLibraryFileAliasView">
- <allow attributes="browserDefault __call__" />
+ <allow attributes="__call__" />
+ <allow interface="zope.publisher.interfaces.browser.IBrowserPublisher" />
   </class>

 </configure>

Changed in launchpad-foundations:
assignee: nobody → Gary Poster (gary)
milestone: 10.01 → 10.02
Gary Poster (gary)
Changed in launchpad-foundations:
status: Triaged → In Progress
Revision history for this message
Diogo Matsubara (matsubara) wrote : Bug fixed by a commit

Fixed in devel r10338 <http://bazaar.launchpad.net/~launchpad-pqm/launchpad/devel/revision/10338>

Changed in launchpad-foundations:
status: In Progress → Fix Committed
Ursula Junque (ursinha)
Changed in launchpad-foundations:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Subscribing...

Other bug subscribers

Remote bug watches

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