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

Bundler plugin says upload successful, when Sentry CLI fails #670

Open
szokeasaurusrex opened this issue Feb 3, 2025 · 3 comments
Open

Comments

@szokeasaurusrex
Copy link
Member

szokeasaurusrex commented Feb 3, 2025

This issue was originally reported by @kubijo in the Sentry CLI repository.

This issue report is based on the information provided in the original issue; I did not try to reproduce the bug myself.

@kubijo, it would be helpful if you could fill in any missing information here, such as the version of the bundler plugin you are using, and any reproduction steps which are missing.

Environment

"@sentry/webpack-plugin": "3.1.1", with Sentry CLI 2.39.1. Running on Ubuntu 24.10.

Steps to Reproduce

  1. Use the bundler plugin to upload sourcemaps to Sentry
  2. When uploading, configure a non-existing project as the destination, so that Sentry CLI fails during the upload.
    • In the original issue, the Sentry CLI failed due to a 502 error. However, triggering any kind of failure in the Sentry CLI should likely cause the same behavior in the bundler plugin.

Expected Result

Bundler plugin logs that an error prevented upload.

Actual Result

Bundler plugin logs that sourcemap upload was successful:

[...]
> Bundled 42 files for upload
> Bundle ID: 95362a95-0674-5519-ad70-4fc1c6bb1d63
error: Project not found. Ensure that you configured the correct project and organization.

[...]

[sentry-webpack-plugin] Info: Successfully uploaded source maps to Sentry

@kubijo
Copy link

kubijo commented Feb 3, 2025

"@sentry/webpack-plugin": "3.1.1",

@getsantry getsantry bot moved this to Waiting for: Product Owner in GitHub Issues with 👀 3 Feb 3, 2025
@kubijo
Copy link

kubijo commented Feb 4, 2025

For some more annoyances...

  • even though I supplied { debug: true } flag in plugin options, it prints Add --log-level=[info|debug] or export SENTRY_LOG_LEVEL=[info|debug] to see more output. on error and I do certainly miss a lot of logging ... you likely do not propagate that to CLI
  • you depend on an old CLI here, can you keep it up to date?

@szokeasaurusrex
Copy link
Member Author

@kubijo we like to keep our issues limited in scope. So, please open a separate issue for the first item you have mentioned.

@lforst has already bumped the Sentry CLI dependency to the latest version in #671. I believe he is planning to release a new version of the bundler plugin today.

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

No branches or pull requests

2 participants