Open Bug 1683646 Opened 4 years ago Updated 1 month ago

webRequest listener can't detect view-source request separately from regular navigation requests

Categories

(WebExtensions :: Request Handling, enhancement, P3)

enhancement

Tracking

(Not tracked)

REOPENED

People

(Reporter: evilpie, Unassigned)

References

Details

Our extension livemarks redirects URLs for RSS feeds to an internal RSS previewer using a moz-extension URL (source). This preview also contains a view-source link for convenience, which we don't want to redirect back to the preview page of course.

Right now it seems to be impossible to detect requests to view-source URLs. The url property just strips the view-source: scheme.

Maybe add details.isViewSource or details.originUrl should be view-source:https://bugzilla.mozilla.org/show_bug.cgi?id=1683646?

Flags: needinfo?(mixedpuppy)

Rather than something specific, lets add a sub_type property that can be an array of strings. This could be useful for future properties if necessary, but we could also use this for fetch which is currently reported as XHR in the type property.

Severity: -- → S3
Type: defect → enhancement
Flags: needinfo?(mixedpuppy)
Priority: -- → P3

Since main_frame and sub_frame filters are commonly used by extensions with the intent to change the content, either by content scripts or stream filters, I would prefer a new ResourceType over a new sub_type property. I see little value in a sub_type to distinguish fetch from XHR, for the lack of use cases where it would make a difference (and in the end, everything can be a fetch, especially with Service Workers that supposedly fetches on behalf of web pages).

See Also: → 1685881

This affected the web extension we use for development and deployment of Wikipedia.org (https://gerrit.wikimedia.org/g/performance/WikimediaDebug/). It has always worked in Chromium (at least since Chromium 61), but did not work in Firefox 67 when we first tested it.

Testing on Firefox 98 today, however, it seems this has been fixed since. We use webRequest.onBeforeSendHeaders.addListener for this purpose.
https://gerrit.wikimedia.org/g/performance/WikimediaDebug/+/refs/tags/2.4.5/background.js#227

Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → DUPLICATE

No, i think this bug was about detecting view-source urls and then not handling these urls?

Flags: needinfo?(evilpies)

(In reply to kernp25 from comment #6)

No, i think this bug was about detecting view-source urls and then not handling these urls?

You're right, I'll re-open.

Comment 4 interpreted the original title differently, so I've renamed the title to clarify it.

Severity: S3 → N/A
Status: RESOLVED → REOPENED
Flags: needinfo?(evilpies)
Resolution: DUPLICATE → ---
See Also: → CVE-2020-35111
Summary: webRequest listener can't detect view-source request → webRequest listener can't detect view-source request separately from regular navigation requests
See Also: → 1828141
See Also: → 1699330
You need to log in before you can comment on or make changes to this bug.