Failed to rewrite frame

[zWingz]

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? If you use the CDN bundles, please specify the exact bundle (e.g. bundle.tracing.min.js) in your SDK setup.

@sentry/node

SDK Version

latest

Framework Version

latest

Link to Sentry event

No response

SDK Setup

No response

Steps to Reproduce

My code ran in the vscode extension and I get the frame.filename is c:\Users\Administrator\.vscode\extensions\xxxxxxx\out\extension.js

It doesn't match the regext /^[A-Z]:\\/, so it can't rewrite the frame

https://github.com/getsentry/sentry-javascript/blob/develop/packages/integrations/src/rewriteframes.ts#L75-L80

Expected Result

rewrite the frame correctly

Actual Result

cannot rewrite

[Lms24]

Hi @zWingz thanks for reporting! I quickly checked the regex and you're right, we're not matching on lower-case windows path prefixes. Will push a PR with the fix soon.

[Lms24]

I opened #7506 to fix this in the default iteratee of RewriteFrames. This will go out in the next release (probably on Monday).

If you need this sooner, you can write your own iteratee with the fix and provide it to the RewriteFrames constructor.

[zWingz]

@Lms24 thanks