fix implementation of 'cache-control: must-revalidate' header

VERIFIED FIXED in mozilla0.9.9

Status

()

P3
normal
VERIFIED FIXED
18 years ago
16 years ago

People

(Reporter: darin.moz, Assigned: darin.moz)

Tracking

Trunk
mozilla0.9.9
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [http/1.1 compliance])

(Assignee)

Description

18 years ago
we currently treat 'cache-control: must-revalidate' as a synonym for
'cache-control: no-cache'.  but according to the http/1.1 spec there is a subtle
difference between the two, and 'must-revalidate' is actually much less strict.

from section 14.9.4:

  When the must-revalidate directive is present in a response received by a   
  cache, that cache MUST NOT use the entry after it becomes stale to respond to 
  a subsequent request without first revalidating it with the origin server.
this simply means that we are permitted to reuse a cached response provided the
cached response is still fresh.  currently, we revalidate the cached response
whenever it is requested (as a 'no-cache' directive would require).

in other words, even if the user has selected a cache validation frequency of
VALIDATE_ONCE_PER_SESSION or VALIDATE_NEVER, we'll still need to revalidate
cached responses (when stale) that contain the 'must-revalidate' control header.

Updated

18 years ago
Blocks: 101502
(Assignee)

Updated

17 years ago
Status: NEW → ASSIGNED
Priority: -- → P3
Target Milestone: --- → mozilla0.9.7
(Assignee)

Updated

17 years ago
Whiteboard: [http/1.1 compliance]
(Assignee)

Updated

17 years ago
Target Milestone: mozilla0.9.7 → mozilla0.9.9
(Assignee)

Updated

17 years ago
Depends on: 112564
(Assignee)

Comment 1

17 years ago
fixed with the landing of the patch for bug 112564.
Status: ASSIGNED → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → FIXED

Comment 2

16 years ago
Verified per darin's comment.
Status: RESOLVED → VERIFIED
QA Contact: tever → junruh
You need to log in before you can comment on or make changes to this bug.