httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Roy T. Fielding" <field...@kiwi.ICS.UCI.EDU>
Subject Re: [PATCH] error responses have wrong headers (part 2)
Date Thu, 08 May 1997 19:07:45 GMT
>> removes a bit of code which is never executed due to r->no_cache never
>> being set when (r->proto_num >= 1001).
>
>I don't think this should be removed. r->no_cache is part of the
>Apache API, mod_negotiation isn't the only module that's allowed to
>set it. Just because mod_neg never sets it for HTTP/1.1+ requests
>doesn't mean that some other module won't, and if it does, then we
>need the Cache-Control: private to be sent, because that would then be
>the correct behavior.

No, we don't, because Cache-Control: private is not the right thing
to send even if r->no_cache is set.  r->no_cache was a temporary hack
that is obsoleted by Vary.  In order to be HTTP/1.1 compliant, such a
module MUST output a correct Vary header field; if it doesn't want to
be HTTP/1.1 compliant, then the Expires field is sufficient.

....Roy

Mime
View raw message