-
-
Notifications
You must be signed in to change notification settings - Fork 228
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
sourcemaps inject
should fall back to co-located source maps if exact path can't be found
#1870
Comments
@brettdh thanks for opening this issue. Might take a while for us to look at it with the holidays coming up. |
Should have mentioned - I'm working on a PR already, as it's a trivial addition to my last one. |
…e a remote path Fixes getsentry#1870.
…e a remote path Fixes getsentry#1870.
…e a remote path Fixes getsentry#1870.
This issue has gone three weeks without activity. In another week, I will close it. But! If you comment or otherwise update it, I will reset the clock, and if you label it "A weed is but an unloved flower." ― Ella Wheeler Wilcox 🥀 |
Sorry, I've let the PR here become idle. I'll try to get back to it soon. |
I put a "Status: Backlog" label on this PR so StaleBot keeps it open |
…e a remote path Fixes getsentry#1870.
Environment
What version of the CLI and what OS are you running?
Sentry CLI 1.23.0, macOS Ventura 13.4
Steps to Reproduce
Similar to #1846:
.js
file with a corresponding.js.map
file in the same directorysentry-cli sourcemaps inject foo.js*
Alternatively, though it was created to repro a different bug, you can also repro this bug by following the steps here, ignoring the AWS bits and running
npm run build
instead of any of the other scripts. Thesentry-cli sourcemaps inject
step will show missing debug IDs for the sourcemap files.Expected Result
JS file and sourcemap file both have debug ID injected
Actual Result
Only JS file has debug ID injected
Logs
When filing a bug report, please attach debug logs, either by using
--log-level=debug
flag orSENTRY_LOG_LEVEL=debug
environment variable.The text was updated successfully, but these errors were encountered: