launchpad-reviewers team mailing list archive

[Colin Watson <cjwatson@xxxxxxxxxxxxx>]

The relevant cache headers are only set for a very limited set of files.
E.g. in ubuntu-mirror:

    <Files ~ "Release(\.gpg)?|InRelease|Packages\.(xz|bz2|gz)|Sources\.(xz|bz2|gz)?|Translation-.*?$">
        ExpiresActive On
        ExpiresDefault "modification plus 1800 seconds"
        Header append Cache-Control "proxy-revalidate"
    </Files>

In general this seems to equate to the core set of files produced by
apt-ftparchive and other necessary publisher functions, rather than the
ones I'm calling "extra" files that are synced in from elsewhere; the
latter would never have had special Expires applied, so all we were
achieving was to bump its Last-Modified, which probably didn't achieve
much except causing unnecessary redownloads.  curl against the current
archive confirms:

  $ curl -Is http://archive.ubuntu.com/ubuntu/dists/xenial/InRelease | egrep '^(Cache-Control|Expires):'
  Cache-Control: max-age=1264, proxy-revalidate
  Expires: Tue, 15 Mar 2016 02:48:00 GMT
  $ curl -Is http://archive.ubuntu.com/ubuntu/dists/xenial/main/dep11/Components-amd64.yml.gz | egrep '^(Cache-Control|Expires):'
  $

Of course this is all fairly weird and ought to be replaced by by-hash,
but in the interim I don't think this change breaks anything to do with
cache headers.

-- 
https://code.launchpad.net/~cjwatson/launchpad/dep11-mtime/+merge/288757
Your team Launchpad code reviewers is subscribed to branch lp:launchpad.