Trailing slash in svn url causes import to fail
Bug #56357 reported by
David Allouche
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Launchpad itself |
Fix Released
|
Medium
|
David Allouche |
Bug Description
Bad formatting of svnrepository sometimes causes cscvs to fail with a mysterious assertion.
See: http://
There should be a database constraint and corresponding input validation to ensure that svnrepository urls are "canonical" by the libsvn definition.
That includes (but is probably not limited to):
* no trailing slash
* no multiple slash
This normalisation must be done at the database level (and not just in cscvs) to prevent circumvention of the unique constraint on svnrepository.
Changed in launchpad-bazaar: | |
importance: | Untriaged → Medium |
status: | Unconfirmed → Confirmed |
description: | updated |
description: | updated |
Changed in launchpad-bazaar: | |
importance: | Medium → High |
Changed in launchpad-bazaar: | |
assignee: | nobody → ddaa |
To post a comment you must log in.
Now, the series/+source form performs the required URL normalization.
This is not ideal, as this should really be a database constraint, but this is good enough to address the visible problem. Making this bug Low importance.