Merge ~cjwatson/lp-archive:always-set-cache-control into lp-archive:main
Proposed by
Colin Watson
Status: | Merged |
---|---|
Approved by: | Colin Watson |
Approved revision: | c84d49cd159f5a24a2df86ae56104281485b2448 |
Merge reported by: | Otto Co-Pilot |
Merged at revision: | not available |
Proposed branch: | ~cjwatson/lp-archive:always-set-cache-control |
Merge into: | lp-archive:main |
Diff against target: |
73 lines (+15/-11) 2 files modified
lp_archive/archive.py (+6/-3) tests/test_archive.py (+9/-8) |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Ines Almeida | Approve | ||
Review via email: mp+455045@code.launchpad.net |
Commit message
Always set Cache-Control header on redirect responses
Description of the change
It's much easier to fix our nginx configuration to copy the Cache-Control header over to the followed-redirect response if we ensure that the redirect always has a Cache-Control header rather than relying on the one returned by the librarian.
Thanks to Clinton Fung for help puzzling this out.
To post a comment you must log in.
LGTM