http redirects break for schemes with implementation name
Bug #853765 reported by
Jelmer Vernooij
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Bazaar |
Fix Released
|
High
|
Jelmer Vernooij | ||
| Launchpad itself |
Fix Released
|
High
|
Jelmer Vernooij | ||
Bug Description
http redirects currently break for redirects that have a target that is not http/https. The path component will be completely dropped.
An example of this can be seen by branching http://
Related branches
lp://staging/~jelmer/bzr/853765-http-redirects-broken
- Vincent Ladeuil: Needs Fixing
-
Diff: 166 lines (+49/-42)3 files modifiedbzrlib/tests/test_http.py (+13/-0)
bzrlib/transport/http/__init__.py (+33/-42)
doc/en/release-notes/bzr-2.5.txt (+3/-0)
lp://staging/~jelmer/launchpad/newer-bzr-http-redirects
- Steve Kowalik (community): Approve (code)
-
Diff: 40 lines (+4/-4)3 files modifiedutilities/sourcedeps.cache (+2/-2)
utilities/sourcedeps.conf (+1/-1)
versions.cfg (+1/-1)
| Changed in launchpad: | |
| status: | New → Confirmed |
| status: | Confirmed → In Progress |
| importance: | Undecided → High |
| assignee: | nobody → Jelmer Vernooij (jelmer) |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #1 |
| tags: | added: qa-needstesting |
| Changed in launchpad: | |
| status: | In Progress → Fix Committed |
| tags: |
added: qa-ok removed: qa-needstesting |
| Changed in bzr: | |
| status: | In Progress → Fix Committed |
| milestone: | none → 2.5b2 |
| status: | Fix Committed → Fix Released |
| Changed in launchpad: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
Fixed in stable r14010 <http://