-
Notifications
You must be signed in to change notification settings - Fork 0
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
Address the timing issue #4
Comments
I have noticed that the proposed solution raises a CSP infringement in some websites such as https://www.vuecinemas.nl/#filmposters. |
Unfortunately none, but a search for CSP violation Privacy Badger or uBlock Origin yields interesting results: |
Just to be clear, I think this is a low priority issue and can be stated as a limitation if a clean workaround cannot be found. What we don't want is to interfere with the websites' CSP policies, for example. Thanks for identifying this bug in the first place! |
I agree, I will give it some thought but it definitely shouldn't delay the wrap-up. As you mention, this is an interesting limitation that a lot of extensions face.
|
We seem to miss early calls to Topics API. Let's follow Privacy Badger's example to solve it:
https://groups.google.com/a/chromium.org/g/chromium-extensions/c/jU3Ib6fP9A8/m/qsLz0HcnAwAJ
https://github.com/EFForg/privacybadger/blob/4b8c4655a4d46c900df99f9837c998bf67d51133/src/js/contentscripts/utils.js#L18-L37
The text was updated successfully, but these errors were encountered: