Launchpad mirror prober considers http redirects broken
Bug #1836712 reported by
Mattias Wadenstein
This bug affects 3 people
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Launchpad itself |
Fix Released
|
High
|
Colin Watson | ||
Bug Description
From a mirror probe run against our mirror I saw messages like this:
Mon Jul 15 10:29:01 2019: Bad response code: 302 on http://
These days apt is happy to follow 302s, and our mirror is by design redirecting large files to offloader hosts for more efficient cache usage (https:/
Related branches
lp:~cjwatson/launchpad/mirror-prober-redirects
- Tom Wardill (community): Approve
-
Diff: 146 lines (+22/-11)5 files modifieddatabase/sampledata/current-dev.sql (+1/-0)
database/sampledata/current.sql (+2/-1)
lib/lp/registry/doc/distribution-mirror.txt (+11/-6)
lib/lp/registry/scripts/distributionmirror_prober.py (+4/-2)
lib/lp/registry/stories/distributionmirror/xx-distribution-mirrors.txt (+4/-2)
lp:~wgrant/launchpad/dmp-redirect-2
- Colin Watson (community): Approve
-
Diff: 36 lines (+14/-1)2 files modifiedlib/lp/registry/doc/distribution-mirror.txt (+13/-0)
lib/lp/registry/scripts/distributionmirror_prober.py (+1/-1)
| tags: | added: lp-registry mirror |
| Changed in launchpad: | |
| assignee: | nobody → Colin Watson (cjwatson) |
| importance: | Undecided → High |
| status: | New → In Progress |
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 launchpad: | |
| status: | Fix Committed → Fix Released |
Revision history for this message
| William Grant (wgrant) wrote | #2 |
| Changed in launchpad: | |
| status: | Fix Released → In Progress |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #3 |
| tags: |
added: qa-needstesting removed: qa-ok |
| Changed in launchpad: | |
| status: | In Progress → Fix Committed |
| tags: |
added: qa-ok removed: qa-needstesting |
| Changed in launchpad: | |
| status: | Fix Committed → Fix Released |
Revision history for this message
| Rogier van Eeten (blauwblaatje) wrote | #4 |
Revision history for this message
| Colin Watson (cjwatson) wrote | #5 |
To post a comment you must log in.
Fixed in stable r19016 <http://