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.

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

Lower permissions. Investigate 'activeTab' 🔒 #129

Closed
alan-null opened this issue Apr 16, 2020 · 1 comment
Closed

Lower permissions. Investigate 'activeTab' 🔒 #129

alan-null opened this issue Apr 16, 2020 · 1 comment
Assignees
@alan-null
Labels
browser-chrome investigation
Milestone
3.4.0

Comments

@alan-null
Copy link
Owner

alan-null commented Apr 16, 2020 •
edited
Loading

Investigate activeTab and check whether we can get rid of hosts tabs

    "permissions": [
        "tabs",
        "http://*/*",
        "https://*/*",
        "storage"
    ],
    "content_scripts": [{
        "matches": [
            "http://*/*",
            "https://*/*"
        ]
    }],
@alan-null alan-null self-assigned this Apr 16, 2020
@alan-null alan-null added this to the 3.4 milestone Apr 16, 2020
@alan-null alan-null changed the title Investigate activeTab permission Lower permissions. Investigate 'activeTab' Apr 22, 2020
@alan-null
Copy link
Owner Author

Turns out that tabs permission is not needed because activeTab is sufficient for the set of API I use in the extension.

@alan-null alan-null changed the title Lower permissions. Investigate 'activeTab' Lower permissions. Investigate 'activeTab' 🔒 Apr 22, 2020
alan-null added a commit that referenced this issue Apr 22, 2020
@alan-null
#129 | Lower permissions 🔒. Investigate 'activeTab'
1c50e6f
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@alan-null alan-null

Labels
browser-chrome investigation
Projects
None yet
Milestone
3.4.0
Development

No branches or pull requests
1 participant
@alan-null