Redirect loop when viewing file

Bug #770008 reported by xaav on 2011-04-24
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Launchpad itself
Undecided
Unassigned
loggerhead
Critical
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.

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
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.

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.

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.

To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Duplicates of this bug

Other bug subscribers