Closed
Bug 86265
Opened 24 years ago
Closed 24 years ago
Only send If-Modified-Since if given a Last-Modified header (do not use the Date header for validation)
Categories
(Core :: Networking: HTTP, defect, P3)
Core
Networking: HTTP
Tracking
()
RESOLVED
FIXED
People
(Reporter: darin.moz, Assigned: darin.moz)
References
Details
(Keywords: topembed, Whiteboard: r=bbaetz, sr=dougt, verified-on-trunk)
Attachments
(1 file)
|
799 bytes,
patch
|
Details | Diff | Splinter Review |
Only send If-Modified-Since if given a Last-Modified header (do not use the Date
header for validation). This would allow servers to control whether or not
mozilla performs conditional requests vs. normal requests.
|
Assignee | |
Updated•24 years ago
|
Target Milestone: --- → mozilla0.9.3
|
|
Assignee | |
Comment 2•24 years ago
|
||
bug 87844 makes this a more serious issue than i previously thought.
Priority: -- → P3
|
|
Assignee | |
Comment 3•24 years ago
|
||
|
|
||
Comment 4•24 years ago
|
||
r=bbaetz
Push this onto the branch as well?
fix is very safe and would help us iron out this issue. Weak cache validators
are very useful for using cached entries. And this may improve our perf on such
pages (such validation cases) I'd recommend getting this onto the branch.
Keywords: nsBranch
|
||
Comment 6•24 years ago
|
||
sr=dougt
|
|
||
Comment 7•24 years ago
|
||
sr=dougt
|
|
Assignee | |
Comment 8•24 years ago
|
||
fix checked in (on the trunk)
Status: NEW → ASSIGNED
Whiteboard: r=bbaetz, sr=?, a=? → r=bbaetz, sr=dougt, checked in on the trunk
|
|
Assignee | |
Updated•24 years ago
|
Whiteboard: r=bbaetz, sr=dougt, checked in on the trunk → r=bbaetz, sr=dougt, fixed on trunk
adding vtrunk keyword to indicate the bug has been fixed on the trunk. Bug left
open for tracking checkin to branch (nsbranch) when appropriate.
Once bug has been fixed on the branch also, pls remove vtrunk keyword.
Keywords: vtrunk
|
|
Assignee | |
Comment 10•24 years ago
|
||
-> gagan (for checkin on the nsbranch)
Assignee: darin → gagan
Status: ASSIGNED → NEW
|
|
||
Comment 11•24 years ago
|
||
verified on trunk - testcases are 01lastmodified and 02lastmodified (internal
only)
WinNT 4 2001070304
Linux rh6 2001063021
Mac os9 200107310
|
|
||
Comment 12•24 years ago
|
||
Missed the branch...
Status: NEW → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
|
|
Assignee | |
Comment 13•24 years ago
|
||
reopening so we can get this checked in on the moz 0.9.2 branch.
Status: RESOLVED → REOPENED
Keywords: topembed
Resolution: FIXED → ---
Whiteboard: r=bbaetz, sr=dougt, fixed on trunk → r=bbaetz, sr=dougt, fixed-on-trunk
|
|
Assignee | |
Updated•24 years ago
|
Target Milestone: mozilla0.9.3 → ---
|
|
||
Updated•24 years ago
|
QA Contact: benc → tever
|
|
||
Updated•24 years ago
|
Whiteboard: r=bbaetz, sr=dougt, fixed-on-trunk → r=bbaetz, sr=dougt, verified-on-trunk
|
|
||
Comment 15•24 years ago
|
||
let's get it on the 0.9.2 branch
|
|
Assignee | |
Comment 16•24 years ago
|
||
fixed-on-branch
Status: NEW → RESOLVED
Closed: 24 years ago → 24 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•