branches with "_" in name cause 404 in loggerhead

Bug #147629 reported by Emil Gurevitch
2
Affects Status Importance Assigned to Milestone
Launchpad itself
Fix Released
Medium
Unassigned

Bug Description

Background info:
I created a new team. I then registered a branch at launchpad.net. I pushed my local bazaar branch to launchpad project. The registrant of the project is the team, not my own account.

Bug:
Whenever I try to "Browse Code" i get a 404 Not Found saying that the /files path was not found.

I tried pushing the branch to a project where I was the registrant, not the team, and then the "Browse code" worked just fint.

Tags: lp-code
Revision history for this message
Michael Hudson-Doyle (mwhudson) wrote :

Which branch is this?

Revision history for this message
Emil Gurevitch (emil-deactivatedaccount) wrote :
Revision history for this message
Michael Hudson-Doyle (mwhudson) wrote :

Ah. This is bug https://bugs.edge.launchpad.net/launchpad-bazaar/+bug/95109 "It is possible to register branches with '_' in their name".

If you rename your branches to not have underscores in the names, codebrowse will work (it is a separate bug that codebrowse does not cope in this situation, but that's due to grotty issues in cherrypy).

Revision history for this message
Tim Penhey (thumper) wrote :

If we know that loggerhead rewrites the urls to avoid underscores, then we should apply that rewrite too on the LP side of things.

Changed in launchpad-bazaar:
importance: Undecided → Medium
status: New → Confirmed
Revision history for this message
Michael Hudson-Doyle (mwhudson) wrote :

I fixed this some time ago. There's still an issue where if there are branches called both foo-bar and foo_bar then you can only browse to one of them, but I think that's a pretty minor problem.

Changed in launchpad-bazaar:
status: Confirmed → 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.