Closed Bug 1700866 Opened 3 years ago Closed 3 years ago

OCSP requests shouldn't use the HTTP cache

Categories

(Core :: Networking: HTTP, defect, P2)

defect

Tracking

()

RESOLVED FIXED
89 Branch
Tracking Status
firefox89 --- fixed

People

(Reporter: valentin, Assigned: valentin)

References

Details

(Whiteboard: [necko-triaged])

Attachments

(1 file)

+++ This bug was initially created as a clone of Bug #1670506 +++

(In reply to Valentin Gosu [:valentin] (he/him) from bug 1670506 comment #10)

I think this issue also affects desktop users in PB mode, where we shouldn't be saving anything persistently, right?
Technically we should be doing the right thing here - we set the originAttributes to the privateBrowsingId.

But sometimes it's possible we don't call UpdatePrivateBrowsing in AsyncOpen.

This bug is about calling UpdatePrivateBrowsing() unconditionally in AsyncOpen to pick up changes to the load info.

See https://bugzilla.mozilla.org/show_bug.cgi?id=1670506#c10
The loadInfo mPrivateBrowsingId value could have changed between the creation
of the channel and calling AsyncOpen, so it's best to call it unconditionally.

Pushed by valentin.gosu@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/86983c54ade3
Call UpdatePrivateBrowsing() unconditionally in AsyncOpen r=necko-reviewers,dragana
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 89 Branch
You need to log in before you can comment on or make changes to this bug.