[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#590478: marked as done (404 errors come with silly cache-control header)

Your message dated Thu, 5 Aug 2010 19:20:35 +0200
with message-id <20100805172035.GT27199@anguilla.noreply.org>
and subject line Re: Bug#590478: 404 errors come with silly cache-control header
has caused the Debian Bug report #590478,
regarding 404 errors come with silly cache-control header
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact owner@bugs.debian.org
immediately.)


-- 
590478: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=590478
Debian Bug Tracking System
Contact owner@bugs.debian.org with problems
--- Begin Message --- 
Package: snapshot.debian.org
Severity: normal

| weasel@intrepid:~$ wget -O /dev/null -S http://snapshot.debian.org/archive/backports.org/20100719T095641Z/dists/lenny-backports/contrib/binary-amd64/Packages.diff/Index
..
|   Cache-Control: no-cache

There's no reason why such a 404 should not be cached, in fact, there's very
good reasons why it should.  Probably want to make it publicly cachable, for at
least a day or so.

--- End Message ---
--- Begin Message --- 
On Mon, 26 Jul 2010, Peter Palfrader wrote:

> Package: snapshot.debian.org
> Severity: normal
> 
> | weasel@intrepid:~$ wget -O /dev/null -S http://snapshot.debian.org/archive/backports.org/20100719T095641Z/dists/lenny-backports/contrib/binary-amd64/Packages.diff/Index
> ..
> |   Cache-Control: no-cache
> 
> There's no reason why such a 404 should not be cached, in fact, there's very
> good reasons why it should.  Probably want to make it publicly cachable, for at
> least a day or so.

Fixed with the latest two commits:

commit eed69ef1f34aa173e655702ae4425fd7eb2b923c
Author: Peter Palfrader <peter@palfrader.org>
Date:   Thu Aug 5 17:17:49 2010 +0000

    Set expire/caching headers earlier in the archive controller so that we send back the default caching time for directories (10minut

commit dc9699875eae1a3a39fd7c526793b499e73e651f
Author: Peter Palfrader <peter@palfrader.org>
Date:   Thu Aug 5 17:17:05 2010 +0000

    On 404 errors, retain the previously set caching headers



weasel@intrepid:~$ date; wget -O /dev/null -S http://snapshot.debian.org/archive/backports.org/20100719T095641Z/dists/lenny-backports/contrib/binary-amd64/Packages.diff/Index 2>&1 | egrep 'Expires|Cache-Contr'
Thu Aug  5 19:20:10 CEST 2010
  Expires: Thu, 05 Aug 2010 17:30:10 GMT
  Cache-Control: max-age=600, public


Yay
-- 
                           |  .''`.  ** Debian GNU/Linux **
      Peter Palfrader      | : :' :      The  universal
 http://www.palfrader.org/ | `. `'      Operating System
                           |   `-    http://www.debian.org/

--- End Message ---
Reply to: