Closed
Bug 47352
Opened 24 years ago
Closed 24 years ago
HTTP 304 response does not redrive HTTPNotify with original headers
Categories
(Core :: Networking: HTTP, defect, P3)
Tracking
()
VERIFIED
FIXED
mozilla0.9.1
People
(Reporter: toml, Assigned: darin.moz)
References
Details
Attachments
(1 file)
From Bugzilla Helper:
User-Agent: Mozilla/4.72 [en] (WinNT; U)
BuildID: 2000073008
When an HTTP 304 response code is received, the original headers associated with
that request are not redriven to the HTTPNotify AsyncExamineResponse
listener/observer. I'm working on the Platform for Privacy Preferences (P3P)
which requires that I see those original response headers since P3P uses
response headers as part of it's protocol.
Reproducible: Always
Steps to Reproduce:
1. Visit a site for the first time.
2. Return to the site receiving a 304 HTTP response.
Actual Results: Only the response headers from the 304 response are sent to the
AsyncExamineResponse HTTPNotify listener.
Expected Results: The response headers from the 304 response should be sent to
the AsyncExamineResponse HTTPNotify listener. Then, the AsyncExamineResponse
HTTPNotify listener should be redriven with the original headers.
Below and attached is a diff with a change that I made to nsHTTPChannel.cpp to
cause the
redrive of the original headers.
Index: nsHTTPChannel.cpp
===================================================================
RCS file: /cvsroot/mozilla/netwerk/protocol/http/src/nsHTTPChannel.cpp,v
retrieving revision 1.213
diff -r1.213 nsHTTPChannel.cpp
2355a2356,2357
> OnHeadersAvailable();
>
Reporter | ||
Comment 1•24 years ago
|
||
Are you sure that's the right thing to do? Just trying to look for sideffects
....
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
pulling in ruslan's necko bugs ->darin
Assignee: ruslan → gagan
Status: ASSIGNED → NEW
Target Milestone: Future → M19
http bugs to "Networking::HTTP"
Assignee: gagan → darin
Component: Networking → Networking: HTTP
Assignee | ||
Updated•24 years ago
|
Target Milestone: --- → Future
Assignee | ||
Updated•24 years ago
|
Assignee | ||
Comment 6•24 years ago
|
||
my changes for bug 76866 will fix this bug as well.
Depends on: 76866
Assignee | ||
Comment 7•24 years ago
|
||
fixed with branch landing
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•