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)

x86
Windows NT
defect

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(); >
-ruslan
Assignee: gagan → ruslan
Are you sure that's the right thing to do? Just trying to look for sideffects ....
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Target Milestone: --- → Future
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
Blocks: 62399
Target Milestone: --- → Future
Severity: blocker → critical
Status: NEW → ASSIGNED
Keywords: nsbeta1, patch
Target Milestone: Future → mozilla0.9.1
my changes for bug 76866 will fix this bug as well.
Depends on: 76866
fixed with branch landing
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
verified
Status: RESOLVED → VERIFIED
Blocks: 96683
No longer blocks: 96683
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: