Expose private browsing flag on proxy/webrequest details

RESOLVED FIXED in Firefox 68

Status

enhancement
RESOLVED FIXED
2 months ago
Last month

People

(Reporter: mixedpuppy, Assigned: mixedpuppy)

Tracking

(Blocks 1 bug, {dev-doc-needed})

unspecified
mozilla68
Dependency tree / graph
Bug Flags:
qe-verify -

Firefox Tracking Flags

(firefox68 fixed)

Details

Attachments

(1 attachment)

Assignee

Description

2 months ago

We want some additional/simplified flags on the details objects received in webrequest and proxy event APIs.

  • private browsing flag
  • container id
  • tracker information (see TrackingClassification[1] and ClassOfService[2])
    needs better definition, probably split out into separate bug

[1] https://searchfox.org/mozilla-central/rev/d33d470140ce3f9426af523eaa8ecfa83476c806/netwerk/protocol/http/HttpTrafficAnalyzer.h#27

[2] https://searchfox.org/mozilla-central/rev/d33d470140ce3f9426af523eaa8ecfa83476c806/netwerk/protocol/http/HttpTrafficAnalyzer.h#21

Bugbug thinks this bug is a task, but please change it back in case of error.

Type: defect → task
Type: task → enhancement

Comment 2

2 months ago

Bug 1391992 is about adding the "cookieStoreId" property to webRequest events (this reflects the container ID).

See Also: → 1391992
Assignee

Comment 3

2 months ago

bug 1391992 will expose the container id

This bug is reduced to the private flag.

Summary: Expose additional details on request events → Expose private browsing flag on proxy/webrequest details
Assignee

Updated

2 months ago
Assignee: nobody → mixedpuppy

Comment 7

2 months ago
Pushed by scaraveo@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/7ba49ce135e6
add incognito flag to proxy and webrequest details r=robwu
Assignee

Updated

2 months ago
Keywords: dev-doc-needed

Comment 8

2 months ago
bugherder
Status: NEW → RESOLVED
Closed: 2 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla68

Comment 9

Last month

Can you please provide some STR for this issue so we can check it manually? If no manual testing is needed please mark it as "qe-verify- "

Flags: needinfo?(mixedpuppy)
Assignee

Comment 10

Last month

covered by tests

Flags: needinfo?(mixedpuppy) → qe-verify-
You need to log in before you can comment on or make changes to this bug.