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

Limit extension's activity to local sites #25

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

t-hofmann
Copy link

This web extension needs to access the content of web pages. This may be a security problem (e. g. reading credentials, falsifying content, ...).

It seems to be good practice to limit itself to only local addresses, - hence this request.

The addresses may match "non-private"-adresses in case an address is constructed in a matching way (e. g. 10.a.b..com), at least it is willing to restrict itself.

Not tested, due to non-available environment.

Description of the matching mechanism:
https://developer.chrome.com/extensions/content_scripts#matchAndGlob
fix: missing js-statement to reload.js, syntax
@sohaibabbas
Copy link

my server is not connect with visual studio

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants