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.

Sign up for GitHub

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

Jump to bottom

Too broad permissions #55

Closed

alexanderkjeldaas opened this issue Jul 18, 2019 · 1 comment

Labels

alexanderkjeldaas commented Jul 18, 2019

Why is this extension asking for permission to access all websites?

This should either be prominently documented in the description or removed.

Owner

thebaer commented Jul 19, 2019

As mentioned here, there are sites powered by Medium that aren't hosted at medium.com:

MMRA/content.js

Lines 85 to 87 in 0e53dcb

    
           // This extension runs on all domains so it can Make Medium Readable Again even for publications on custom domains. 
        
           // Here we make sure the code only runs on Medium sites. 
        
           if (document.querySelector('head meta[property="al:ios:app_name"][content="medium" i]')) {

This is the correct solution: #15. Closing since it's a duplicate.

thebaer closed this as completed

thebaer added the duplicate label

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment