fix: Handle custom error names in TraceKit#1237
Conversation
vendor/TraceKit/tracekit.js
Outdated
|
|
||
| var chrome = /^\s*at (.*?) ?\(((?:file|https?|blob|chrome-extension|native|eval|webpack|<anonymous>|[a-z]:|\/).*?)(?::(\d+))?(?::(\d+))?\)?\s*$/i, | ||
| gecko = /^\s*(.*?)(?:\((.*?)\))?(?:^|@)((?:file|https?|blob|chrome|webpack|resource|\[native).*?|[^@]*bundle)(?::(\d+))?(?::(\d+))?\s*$/i, | ||
| gecko = /^\s*(.*?)(?:\((.*?)\))?(?:^|@)((?:file|https?|blob|chrome|webpack|resource).*?:\/.*?|\[native code\]|[^@]*bundle)(?::(\d+))?(?::(\d+))?\s*$/i, |
There was a problem hiding this comment.
pretty sure blob is blob:foo and not blob://fo
There was a problem hiding this comment.
Blob is blob:http://url/for/some/endpoint, which is matched by blob.*?:\/:) We have a test for it in place
There was a problem hiding this comment.
That depends on the browser. Current chrome adds the http origin to it, but blob:UUID is perfectly permissible and shows up in Firefox for instance.
You can try with (URL.createObjectURL(new Blob([new Int8Array()]))).toString()
There was a problem hiding this comment.
Apparently blob urls are now supposed to always have an origin. Since they are rare I might be okay with that but please add a comment why blob is okay.
There was a problem hiding this comment.
Added appropriate note and reformatted variables slightly so it's easier to move around.
There was a problem hiding this comment.
Also made it two separate commits so it's easier to track precise changes
14489ea to
1cc1915
Compare
1cc1915 to
845c83b
Compare
Fixes Dependabot alerts #1236 (SSE Event Injection) and #1237 (Path Traversal via Double Decoding) by updating the yarn.lock resolution for h3 from 1.15.5 to 1.15.10. Easier fix than #19910 as Nuxt 3.21.x introduced breaking type changes (removed nitro from NuxtOptions and nitro:config from NuxtHooks). Co-authored-by: Claude Sonnet 4 <noreply@anthropic.com>
Fixes Dependabot alerts #1236 (SSE Event Injection) and #1237 (Path Traversal via Double Decoding) by updating the yarn.lock resolution for h3 from 1.15.5 to 1.15.10. Easier fix than #19910 as Nuxt 3.21.x introduced breaking type changes (removed nitro from NuxtOptions and nitro:config from NuxtHooks). Co-authored-by: Claude Sonnet 4 <noreply@anthropic.com>
Fixes #1209 (comment)