Closed Bug 1714299 Opened 6 months ago Closed 5 months ago

Correct ORB's opaque response implementation

Categories

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

defect

Tracking

()

RESOLVED FIXED
91 Branch
Tracking Status
firefox91 --- fixed

People

(Reporter: annevk, Assigned: edenchuang)

References

(Blocks 1 open bug, Regressed 1 open bug)

Details

(Whiteboard: [necko-triaged])

Attachments

(1 file)

As discussed offline the telemetry for ORB is somewhat off due to incorrectly identifying opaque responses. We should correct this and then observe if the telemetry becomes more in line with what would be expected.

In particular, an opaque response depends on request's mode (has to be "no-cors"), and depends on whether there was ever a cross-origin response to the request (i.e., a redirect chain of A -> B -> A results in an opaque response).

Severity: -- → S3
Priority: -- → P2
Whiteboard: [necko-triaged]
Severity: S3 → S2
Status: NEW → ASSIGNED
Priority: P2 → P1
Whiteboard: [necko-triaged]
Severity: S2 → S3
Priority: P1 → P2
Whiteboard: necko-triaged
Attachment #9225734 - Attachment description: Bug 1714299 - Fix the opaque response judgment for opaque response blocking. → Bug 1714299 - Correct the opaque response judgment for opaque response blocking.
Whiteboard: necko-triaged → [necko-triaged]
Pushed by echuang@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/5e9693e4d6b7
Correct the opaque response judgment for opaque response blocking. r=necko-reviewers,annevk,dragana
Regressions: 1717629

Backed out for causing high frequency failures on browser_opaque_response_blocking_telemetry.js

[task 2021-06-22T14:55:01.998Z] 14:55:01     INFO - TEST-PASS | netwerk/test/browser/browser_opaque_response_blocking_telemetry.js | Should not have recorded key Allowed_SniffAsImageOrAudioOrVideo - 
[task 2021-06-22T14:55:01.999Z] 14:55:01     INFO - Fetching cross-origin url: https://example.com/browser/netwerk/test/browser//res_object.html.
[task 2021-06-22T14:55:01.999Z] 14:55:01     INFO - Validating if the telemetry probe has been reported.
[task 2021-06-22T14:55:02.000Z] 14:55:01     INFO - Buffered messages finished
[task 2021-06-22T14:55:02.001Z] 14:55:02     INFO - TEST-UNEXPECTED-FAIL | netwerk/test/browser/browser_opaque_response_blocking_telemetry.js | Should not have recorded key Allowed_SniffAsImageOrAudioOrVideo - 
[task 2021-06-22T14:55:02.001Z] 14:55:02     INFO - Stack trace:
[task 2021-06-22T14:55:02.001Z] 14:55:02     INFO - chrome://mochikit/content/browser-test.js:test_ok:1331
[task 2021-06-22T14:55:02.001Z] 14:55:02     INFO - chrome://mochitests/content/browser/netwerk/test/browser/browser_opaque_response_blocking_telemetry.js:null:225
[task 2021-06-22T14:55:02.216Z] 14:55:02     INFO - Fetching cross-origin url: https://example.org/browser/netwerk/test/browser/res_img.png.
[task 2021-06-22T14:55:02.241Z] 14:55:02     INFO - Validating if the telemetry probe has been reported.
[task 2021-06-22T14:55:02.242Z] 14:55:02     INFO - TEST-PASS | netwerk/test/browser/browser_opaque_response_blocking_telemetry.js | Should not have recorded key Allowed_SniffAsImageOrAudioOrVideo -
Flags: needinfo?(echuang)
Pushed by echuang@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/f694fff24c46
Correct the opaque response judgment for opaque response blocking. r=necko-reviewers,annevk,dragana
Status: ASSIGNED → RESOLVED
Closed: 5 months ago
Resolution: --- → FIXED
Target Milestone: --- → 91 Branch
Flags: needinfo?(echuang)
You need to log in before you can comment on or make changes to this bug.