New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 883586 link

Starred by 1 user

Issue metadata

Status: Duplicate
Merged: issue 884932
Owner:
Closed: Sep 17
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug



Sign in to add a comment

Prevent requests made by omnibox, ntp doodle from being visible to extensions.

Project Member Reported by karandeepb@chromium.org, Sep 13

Issue description

Extensions can intercept some requests made by the NTP, Omnibox, e.g.
- https://www.google.com/async/ddljson?async=ntp:1
- https://www.google.com/complete/search?client=chrome-omni....

using the web request API.
 
Will this be automagically fixed when we require permission to access the initiator on the webRequest API?  (Since the initiator here should either be absent or the NTP, I'd hope?)
Currently they don't have an initiator. (I found these by logging requests without an initiator). So the fix is probably to add explicit initiators to these requests. We do already prevent extensions from intercepting requests with chrome://newtab as the initiator.

As regards to this getting fixed with the change to the web request api, yes and no. We do still need to add explicit initiators for that to work. But yeah blacklisting initiators like chrome://newtab would then not be necessary.
Are there cases in which network requests don't have initiators and we want to expose them to the webRequest API?  Or could we default to "no initiator == no access"?
This probably belongs in crbug.com/157736. One case is browser initiated main frame requests, but those can be special cased. I am not sure under what cases can renderer initiated requests not have an initiator. Other than that is the case of non-navigation browser initiated requests which don't have an initiator. Off the top of my head, I can't think of cases where an extension should want to intercept those.
Mergedinto: 884932
Status: Duplicate (was: Assigned)
Closing this. Will be tracked as part of  crbug.com/884932 .
Project Member

Comment 6 by sheriffbot@chromium.org, Jan 11

Labels: -Restrict-View-SecurityTeam allpublic
This bug has been closed for more than 14 weeks. Removing security view restrictions.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Sign in to add a comment