-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Sentry code triggering violation from Google manifest v3 during publication #14579
Comments
We need more information to debug this:
|
Yes it is a browser extension listed in the Chrome Web Store. As of their manifest v3 restrictions it has flagged the mentioned points. Unfortunately the only way to reproduce is to resubmit the extension to the web store and wait for the team to audit the update and either accept or reject. Here is an example of a similar issue faced by another library. |
This is likely a duplicate of #14010. I'll close this in favor of that issue, feel free to add more thoughts there. TLDR: This should be tree-shaken if you do not use it. What exact config/init is used, and what build config? This likely needs to be tree shaken out, so maybe something has to be configured in the bundler or similar 🤔 |
Is there an existing issue for this?
How do you use Sentry?
Sentry Saas (sentry.io)
Which SDK are you using?
@sentry/browser
SDK Version
8.9.1
Framework Version
No response
Link to Sentry event
No response
Reproduction Example/SDK Setup
No response
Steps to Reproduce
NA
Expected Result
No violation is to be triggered when using the NPM package as described here: https://docs.sentry.io/platforms/javascript/best-practices/shared-environments/
Actual Result
Violation report:
It seems to be from this file: https://github.com/getsentry/sentry-javascript/blob/develop/packages/browser/src/utils/lazyLoadIntegration.ts
The text was updated successfully, but these errors were encountered: