launchpad.branches.getByUrl() doesn't support properly urlencoded URLs
Bug #704606 reported by
Jelmer Vernooij
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Bazaar |
Triaged
|
Medium
|
Unassigned | ||
| Launchpad itself |
Invalid
|
High
|
Unassigned | ||
| Ubuntu Distributed Development |
Invalid
|
Undecided
|
Unassigned | ||
Bug Description
launchpad.
e.g.:
(Pdb) print launchpad.
https:/
(Pdb) print launchpad.
None
(Pdb) print launchpad.
https:/
This breaks lp-propose, which uses the urlencoded URL.
| summary: |
- confusing error from lp-propose when a MP exists + lp-propose does not work against packaging branches |
| description: | updated |
| tags: | added: udd |
| summary: |
- lp-propose does not work against packaging branches + launchpad.branches.getByUrl() doesn't support properly urlencoded URLs |
| Changed in udd: | |
| status: | New → Invalid |
| description: | updated |
| tags: | removed: udd |
| tags: | added: api |
| Changed in launchpad: | |
| importance: | Medium → High |
Revision history for this message
| Robert Collins (lifeless) wrote | #1 |
| Changed in launchpad: | |
| status: | Triaged → Invalid |
Revision history for this message
| Jelmer Vernooij (jelmer) wrote | #2 |
| Changed in bzr: | |
| status: | New → Triaged |
| importance: | Undecided → Medium |
| tags: | added: check-for-breezy |
To post a comment you must log in.
Per
pchar = unreserved / pct-encoded / sub-delims / ":" / "@"
path element canonical form includes literal + - pct escaping them is incorrect and not canonical. The overview of the standard (std66) mentions that pct encoding where it is not needed will change the interpretation of urls, which is the case here.