Skip to content

Commit

Permalink
Merge pull request #728 from oliverdunk/site-access-request-rename
Browse files Browse the repository at this point in the history
Update `addSiteAccessRequest` proposal to use `addHostAccessRequest` name
  • Loading branch information
oliverdunk authored Nov 22, 2024
2 parents ead0663 + 2e7723e commit 10d6ae6
Show file tree
Hide file tree
Showing 3 changed files with 36 additions and 36 deletions.
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
# permissions.[add/remove]SiteAccessRequest() API
# permissions.[add/remove]HostAccessRequest() API

## API Overview

Expand All @@ -9,69 +9,69 @@ Extensions can request host permissions via:
- Manifest declared `optional_host_permissions`, granted at runtime after a user gesture by `<browser>.permissions.request()`
- Manifest declared `matches` for a `content_script`, granted at install time by default

In some browsers, users can withhold host permissions causing the extension to only have site access when the extension is invoked. For example, in Chrome a user can set an extension to run only "on click". When the extension is clicked, it gains site access to the tab's main frame origin (effectively acting like [`activeTab` permission](https://developer.chrome.com/docs/extensions/develop/concepts/activeTab#what-activeTab-allows).
In some browsers, users can withhold host permissions causing the extension to only have host access when the extension is invoked. For example, in Chrome a user can set an extension to run only "on click". When the extension is clicked, it gains access to the tab's main frame origin (effectively acting like [`activeTab` permission](https://developer.chrome.com/docs/extensions/develop/concepts/activeTab#what-activeTab-allows).

Each browser may decide how it signals the user when an extension is requesting access (e.g through the extensions menu). However, there is no way for an extension to explicitly signal at runtime it’s requesting site access after it was withheld without a user gesture and a heavy-weight permission dialog.
Each browser may decide how it signals the user when an extension is requesting access (e.g through the extensions menu). However, there is no way for an extension to explicitly signal at runtime it’s requesting host access after it was withheld without a user gesture and a heavy-weight permission dialog.


### Objective

Allow the extension to show site access requests at runtime without any user gesture in a less-obtrusive way than with `permissions.request()`. This can be done with a new API that:
Allow the extension to show host access requests at runtime without any user gesture in a less-obtrusive way than with `permissions.request()`. This can be done with a new API that:

- Applies to a specific tab or document id. Developers can specify inactive tabs (request will be shown by the browser when the user activated them).
- Doesn’t need to be made inside the handler for a user action
- Shows the request in the UI, handled differently by each browser. See more in [UI Elements and User-Visible Effects](#ui-elements-and-user-visible\-effects) section
- When accepted, grants always access to the site’s top origin
- When accepted, grants always access to the host
- Resets the request on cross-origin navigation

#### Use Cases

An extension requested access to a site but the user withheld its access and forgot about it. Extension wants to signal it needs site access to execute without user action. For example, “shopping” extension wants to show site access when user navigates to “amazon.com” and access was withheld.
An extension requested access to a host but the user withheld its access and forgot about it. Extension wants to signal it needs host access to execute without user action. For example, “shopping” extension wants to show host access when user navigates to “amazon.com” and access was withheld.

### Consumers

Extensions that want to signal the user when they need site access.
Extensions that want to signal the user when they need host access.

## Implementation

### API Schema

```
// Adds a site access request. Request will only be signaled to the user if
// extension can be granted access to site in the request (i.e., one specified
// Adds a host access request. Request will only be signaled to the user if
// extension can be granted access to host in the request (i.e., one specified
// in host_permissions, optional_host_permissions, a content script match
// pattern, or an applicable activeTab site).
// pattern, or an applicable activeTab host).
// Resolves whether the request is valid.
Promise<boolean> browser.permissions.addSiteAccessRequest({
// The id of the tab where site access requests can be shown. If provided,
Promise<boolean> browser.permissions.addHostAccessRequest({
// The id of the tab where host access requests can be shown. If provided,
// the request is shown on the specified tab and is removed when the tab
// navigates to a new origin.
// Chrome requires either this or `documentId` to be specified.
number?: tabId,
// The id of a document where site access requests can be shown. Must be
// The id of a document where host access requests can be shown. Must be
// the top-level document within a tab. If provided, the request is shown on
// the tab of the specified document and is removed when the document
// navigates to a new origin.
// Chrome requires either this or `tabId` to be specified.
string?: documentId,
// The URL pattern where site access requests can be shown. If provided,
// site access requests will only be shown on URLs that match this pattern.
// The URL pattern where host access requests can be shown. If provided,
// host access requests will only be shown on URLs that match this pattern.
// Browsers may require different levels of specificity.
string?: pattern,
},
callback?: function,
);
// Removes a site access request, if existent.
// Removes a host access request, if existent.
// Resolves whther the request was removed.
Promise<boolean> browser.permissions.removeSiteAccessRequest({
// The id of the tab where site access request will be removed.
Promise<boolean> browser.permissions.removeHostAccessRequest({
// The id of the tab where host access request will be removed.
// Chrome requires either this or `documentId` to be specified.
number?: tabId,
// The id of a document where site access request will be removed.
// The id of a document where host access request will be removed.
// Chrome requires either this or `tabId` to be specified.
string?: documentId,
// The URL pattern where site access request will be removed.
// The URL pattern where host access request will be removed.
string?: pattern,
},
callback?: function,
Expand All @@ -83,12 +83,12 @@ Note: We don’t support iframes since they are not included in the runtime host
### New Permissions
| Permission Added | Proposed Warning |
| ---------------- | -------------------------------------------------------- |
| N/A | Permissions API is used to “request declared optional permissions at run time rather than install time, so users understand why the permissions are needed and grant only those that are necessary” (according to documentation). The goal of this new method is for the extension to request site access, which is effectively a permission. We can adjust the description to not be restricted just to optional permissions. |
| N/A | Permissions API is used to “request declared optional permissions at run time rather than install time, so users understand why the permissions are needed and grant only those that are necessary” (according to documentation). The goal of this new method is for the extension to request host access, which is effectively a permission. We can adjust the description to not be restricted just to optional permissions. |

#### Other Alternatives considered

- Action API is used to control the extension’s button in the browser’s toolbar. It’s exposed if the extension includes the "action" key in the manifest. This is troublesome since an extension could not have an action, but still want to show site access requests. We should not limit requests for extensions with actions
- `permissions.addSiteAccessRequest()` resolves when request is accepted/rejected. An extension could be requesting site access and be granted site access through another mechanism (e.g changing site access in the extensions menu). We would either return a) true if we consider permissions granted through other mechanisms or b) false, because permission wasn't explicitely granted through the request. a) adds complexity and b) may cause confusion. Thus, we consider better to resolve whether the request is valid, and separately extension can listen whether permission is granted through `permissions.onAdded()`.
- Action API is used to control the extension’s button in the browser’s toolbar. It’s exposed if the extension includes the "action" key in the manifest. This is troublesome since an extension could not have an action, but still want to show host access requests. We should not limit requests for extensions with actions
- `permissions.addHostAccessRequest()` resolves when request is accepted/rejected. An extension could be requesting host access and be granted host access through another mechanism (e.g changing host access in the extensions menu). We would either return a) true if we consider permissions granted through other mechanisms or b) false, because permission wasn't explicitely granted through the request. a) adds complexity and b) may cause confusion. Thus, we consider better to resolve whether the request is valid, and separately extension can listen whether permission is granted through `permissions.onAdded()`.

### Manifest Changes

Expand All @@ -102,11 +102,11 @@ None.

#### Persistence

Showing a site access request is never persisted.
Showing a host access request is never persisted.

#### Alignment with Other Vendors and Action APIs

Each browser can decide how they want to signal the extension site access requests. Browsers may decide to ignore the request (e.g due to noisiness, or if an extension has no visible UI).
Each browser can decide how they want to signal the extension host access requests. Browsers may decide to ignore the request (e.g due to noisiness, or if an extension has no visible UI).
On Chrome’s side, we are exploring adding an ‘Allow’ chip in the extensions toolbar.

## User Experience
Expand All @@ -115,29 +115,29 @@ On Chrome’s side, we are exploring adding an ‘Allow’ chip in the extension

Each browser will handle the request as desired. On Chrome’s side, we are considering showing an ‘Allow’ button in the toolbar for these requests. On click, it would grant persistent access to the site’s origin (which can be again withheld by the user)

!["Site access request on Chrome toolbar"](./assets/permissions-requestSiteAccess-chrome-toolbar.png)
!["Host access request on Chrome toolbar"](./assets/permissions-addHostAccessRequest-chrome-toolbar.png)

We would also provide a way for users to silence the requests for an extension through a setting in the extensions menu and in chrome://extensions.

!["Extensions menu on Chrome"](./assets/permissions-requestSiteAccess-chrome-menu.png)
!["Extensions menu on Chrome"](./assets/permissions-addHostAccessRequest-chrome-menu.png)

#### Attribution
The site access request will be attributed directly to the extension.
The host access request will be attributed directly to the extension.

## Security and Privacy

### Exposed Sensitive Data

This API does not directly expose any sensitive data to the extension. However, it could lead to the extension gaining access to the site.
This API does not directly expose any sensitive data to the extension. However, it could lead to the extension gaining access to the host.

### Abuse Scenarios

Extensions can enable site access requests on every tab, which could be annoying to the user.
Extensions can enable host access requests on every tab, which could be annoying to the user.

#### Mitigations
Attacks of annoyance are not in our threat model. Extensions can have annoying behaviors, but they risk uninstallation.
On the browser side, we can place some restrictions to limit noisiness:
- User can decide whether they want to see site access requests for an specific extension
- User can decide whether they want to see host access requests for an specific extension
- We can explore setting a max amount of times the request is shown when the user ignores it.

### Additional Security Considerations
Expand All @@ -147,17 +147,17 @@ None

### Existing Workarounds

Users can change an extension's site access. The only way for extensions to signal the user they want to regain access to the site is through `permissions.request()` flow which is noisy and requires a user gesture. Otherwise, they rely on the user realizing through the browser UI (e.g extensions menu).
Users can change an extension's host access. The only way for extensions to signal the user they want to regain access to the host is through `permissions.request()` flow which is noisy and requires a user gesture. Otherwise, they rely on the user realizing through the browser UI (e.g extensions menu).

Extensions have no way to signal the user that they want access to the site, and rely on the user realizing through the browser UI (e.g extensions menu)
Extensions have no way to signal the user that they want access to the host, and rely on the user realizing through the browser UI (e.g extensions menu)

### Other Alternatives Considered

Specifying URL patterns in the extension's manifest. We decided against that because:
- This is designed to be a highly-contextual signal. The extension should do it only if they have strong believe they will provide value to the user on the given page. This should not be a passive, "hey, I think I can do something here", it should be a "hey, you, the user, probably want me to do something here".
- We do not want extensions to simply show a request on every page, and specifying a list of patterns would lend itself to that behavior (even by not allowing broad match patterns).
- It's too close to host permissions themselves. We suspect that the vast majority of extensions would just have the same field match as in their host permissions, since there is no more knowledge at manifest time about why the extension would run on a specific site.
That being said, each browser a URL pattern can be specified in `permissions.addSiteAccessRequest`. Each browser can determine whether other parameters must be specified apart from the URL pattern.
- It's too close to host permissions themselves. We suspect that the vast majority of extensions would just have the same field match as in their host permissions, since there is no more knowledge at manifest time about why the extension would run on a specific host.
That being said, each browser a URL pattern can be specified in `permissions.addHostAccessRequest`. Each browser can determine whether other parameters must be specified apart from the URL pattern.

### Open Web API
This is related to the extensions showing site access requests in the browser; it doesn't affect the web.
This is related to the extensions showing host access requests in the browser; it doesn't affect the web.

0 comments on commit 10d6ae6

Please sign in to comment.