Support ws and wss protocols on the webRequest API

RESOLVED FIXED in Firefox 55

Status

()

Toolkit
WebExtensions: Request Handling
P2
normal
RESOLVED FIXED
a month ago
15 days ago

People

(Reporter: rpl, Assigned: mixedpuppy)

Tracking

(Blocks: 1 bug)

unspecified
mozilla55
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox55 fixed)

Details

(Whiteboard: [triaged])

MozReview Requests

()

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(1 attachment)

(Reporter)

Description

a month ago
WebExtensions addons may want to listen for webRequest related to websocket connections (which use the ws and wss as the protocol names), e.g. uBlock-Origin is one of these extensions:

- https://github.com/gorhill/uBlock/blob/ca32a693df510fb1018ecde2e8a2412f622aab3c/platform/chromium/vapi-background.js#L1099-L1119

Currently, when trying to add a webRequest listener using ws or wss urls in the event filters logs, the following errors are logged in the console:

- Invalid match pattern: 'ws://*/*'
- Invalid match pattern: 'wss://*/*'
- The webRequest.addListener filter doesn't overlap with host permissions
(Reporter)

Updated

a month ago
Blocks: 1309926

Updated

26 days ago
Assignee: nobody → mixedpuppy
Priority: -- → P2
Whiteboard: [triaged]
(Assignee)

Comment 1

26 days ago
Websockets work fine in webrequest, the urls show up under http/s.  We could change the url for addons listening on websockets, but I wonder where else that url could end up being an issue.

In any case this waits on bug 1368102
Depends on: 1368102
Comment hidden (mozreview-request)
(Assignee)

Updated

17 days ago
Attachment #8875399 - Flags: review?(lgreco)

Comment 3

16 days ago
mozreview-review
Comment on attachment 8875399 [details]
Bug 1367478 support websocket ws/wss protocol in matchpattern,

https://reviewboard.mozilla.org/r/146840/#review151538
Attachment #8875399 - Flags: review?(kmaglione+bmo) → review+

Comment 4

16 days ago
Pushed by mixedpuppy@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/62e44a827308
support websocket ws/wss protocol in matchpattern, r=kmag

Comment 5

15 days ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/62e44a827308
Status: NEW → RESOLVED
Last Resolved: 15 days ago
status-firefox55: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
(Reporter)

Updated

15 days ago
Attachment #8875399 - Flags: review?(lgreco)
You need to log in before you can comment on or make changes to this bug.