Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Why does this extension need "Access your data for all websites"? #276

Open
improwise opened this issue Jan 22, 2025 · 4 comments
Open

Why does this extension need "Access your data for all websites"? #276

improwise opened this issue Jan 22, 2025 · 4 comments
Labels
documentation Improvements or additions to documentation

Comments

@improwise
Copy link

improwise commented Jan 22, 2025

Have tried to find the answer to this by searching but not been able to find it. How come this extension needs access to "Access your data for all websites"? This including usernames and passwords AFAIK makes this a quite sensitive setting.

I also do not see that permission being mentioned here:

Permissions

Mullvad Browser Extension requires the following permissions:

    management to be able to recommend third party extensions
    privacy to disable webRTC and check HTTPS-Only status
    storage to save preferences
    search to recommend other search engines
    *://*.mullvad.net/* to get proxy servers list and display your connection information (See Network requests for details)

The following permissions are optional, but are needed to use the proxy feature:

    proxy to configure and use Mullvad proxy servers
    tabs to show proxy settings from active tab
    <all_urls> to have granular proxy settings

@ruihildt ruihildt added the documentation Improvements or additions to documentation label Jan 23, 2025
@ruihildt
Copy link
Member

ruihildt commented Jan 23, 2025

I tried to convey this with <all_urls> to have granular proxy settings, but it seems it was not clear.

I can probably do better with the one line explanation. What do you think about the following?

<all_urls> to intercept all requests, in order to be able to specify a proxy configuration per domain

@ruihildt
Copy link
Member

In the previous proxy implementation, we set the proxy for the whole browser. (the equivalent of setting the proxy manually through Menu > Settings > General > Network settings)

With the newest implementation (from 0.9.0), we intercept every request and then decide whether to proxy it based on the domain.

Intercepting every request requires the <all_urls> permission.

@ruihildt ruihildt moved this from Triage to In Progress in Mullvad Browser Extension Jan 23, 2025
@improwise
Copy link
Author

I tried to convey this with <all_urls> to have granular proxy settings, but it seems it was not clear.

I can probably do better with the one line explanation. What do you think about the following?

<all_urls> to intercept all requests, in order to be able to specify a proxy configuration per domain

Must admit that I am not up to date with permissions these days but is "Access your data for all websites" really needed to access URLs or is there a more limited way of getting that without getting all the rest?

If the "Access your data for all websites" permission is really required, I think it needs to be mentioned a bit more clearly why and perhaps also in layman terms what it means. That said, I can also see how that could perhaps trigger concern that isn't warranted due to the fact that I assume that nothing more than accessing the URLs is actually used in the extension. Then of course we have people like me more concerned about finding that that permission was needed without it being clearly mentioned or me understanding why (to begin with).

Thanks.

@ruihildt
Copy link
Member

Must admit that I am not up to date with permissions these days but is "Access your data for all websites" really needed to access URLs or is there a more limited way of getting that without getting all the rest?

Yes it is needed. Permissions can be sometimes too broad for the use case, but nothing we can do about it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
Status: In Progress
Development

No branches or pull requests

2 participants