Redirect loop when viewing file

Bug #770008 reported by xaav
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Launchpad itself
Invalid
Undecided
Unassigned
loggerhead
Triaged
Critical
Unassigned
loggerhead-breezy
Triaged
High
Unassigned

Bug Description

When clicking on a link to view a file (example: http://bazaar.launchpad.net/~tsep-dev/tsep/2.0-dev/files/head:/web/app.php?file_id=app-20110423205850-npe15bktb5xalmnb-8), sometimes the file will cause a redirect loop.

Revision history for this message
j.c.sackett (jcsackett) wrote :

See also bug 741783, which may be a duplicate of this.

Changed in launchpad:
status: New → Triaged
importance: Undecided → Critical
Changed in loggerhead:
status: New → Triaged
Changed in launchpad:
status: Triaged → Invalid
importance: Critical → Undecided
Changed in loggerhead:
importance: Undecided → Critical
Revision history for this message
xaav (xaav) wrote :

This problem occurs when trying to view a file, the other problem occurs when trying to view the repository.

However, the same code could be causing both problems.

Revision history for this message
asdfghjkl (asdfghjkl-deactivatedaccount1) wrote :

When viewing a directory in loggerhead the url path can designate either the revno_url and absolutepath for the directory - or the file_id for the directory. When the file_id is used (as with breadcrumb links), the file listing is rendered with incorrect urls (the absolutepath of the *directory* is provided as the file_id for each *file*):

e.g.

http://127.0.0.1:8080/files/head:/someproject/src/main.py?file_id=src-20110504123105-ijtcm0g7a3txui1b-6

the view controller then interprets the file as a directory (based on the incorrect file_id) and passes it back to the inventory controller, which I presume passes it back to the view controller, and so on..

I don't know how best to fix it (this is my first look at the loggerhead source), or if it's already been fixed.. I thought I'd write this anyway.

Revision history for this message
Martin Pool (mbp) wrote :

ftr bug 741783 is not a dupe, or at least not obviously a dupe, so let's leave it separate and fixed.

Jelmer Vernooij (jelmer)
Changed in loggerhead-breezy:
status: New → Triaged
importance: Undecided → High
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

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