Description
Is there an existing issue for this?
- I have checked for existing issues https://github.com/getsentry/sentry-javascript/issues
- I have reviewed the documentation https://docs.sentry.io/
- I am using the latest SDK release https://github.com/getsentry/sentry-javascript/releases
How do you use Sentry?
Sentry Saas (sentry.io)
Which SDK are you using?
@sentry/browser
SDK Version
8.47.0
Framework Version
18.3.1
Link to Sentry event
No response
Reproduction Example/SDK Setup
As you kindly asked in the previous issue, #14010 (comment), I created a small repo with a reproduction to build a Chrome Extension for the side panel:
https://github.com/FedorT22/sandbox-chrome-extension
Steps to Reproduce
- Build the project with @sentry
feedbackIntegration
function using@sentry/browser
or@sentry/react
. You can followREADME.md
file in the project. - See in the
/dist/sidePanel.js
anddist/sidePanel.js.map
the piece of code wtih CDN link:
const baseURL = (options && options.cdnBaseUrl) || 'https://browser.sentry-cdn.com';\n\n return new URL(`/${SDK_VERSION}/${bundle}.min.js`, baseURL).toString();\n
- Submit the extension to the Chrome Webstore
- Wait for the Chrome Webstore answer
Expected Result
Extension is approved by the reviewers.
Actual Result
Extension is rejected by the reviewers
The given reason:
Violation reference ID: Blue Argon
Technical Requirements - Additional Requirements for Manifest V3:
Violation:
Including remotely hosted code in a Manifest V3 item.
Violating Content:
Code snippet: sidePanel.js: "function $re(e) { const t = an(), n = t && t.getOptions(), r = n && n.cdnBaseUrl || "https://browser.sentry-cdn.com/"; return new URL(/${mc}/${e}.min.js
, r).toString() }"
The current work-around:
Clear up @sentry CDN links during the build process with a help of patch-package. See more info in the previous issue.
Metadata
Metadata
Assignees
Type
Projects
Status