-
Notifications
You must be signed in to change notification settings - Fork 17
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
Hardening: Remove or at least warn about sections.url #29
Comments
I would go even further and advocate against the use of indexed source maps. It's not much harder for a tool to rewrite source maps properly today. |
Do any tools support this [edit: specifically, the If this feature doesn't match implementations, we should remove it. |
@mitsuhiko We're actively using index maps in general (no |
@mitsuhiko Let's break out the discussion about indexed source maps into a separate issue then? Here, let's focus on the |
Chromium DevTools supports index maps (and has always supported them), but does not support the Regarding tools: @sokra recently filed an issue regarding performance of index maps in Chrome DevTools, which might indicate that webpack and/or turbopack might be using them as well soon(ish)? 😄 |
I separated out the wider discussion about the index maps. |
I can confirm the same for https://github.com/mozilla/source-map which is used by Firefor DevTools ( and maybe used by other tools) that indexed sourcemaps are supported but not the |
Index maps in the spec support putting each section into a separate resource, linking via the
url
field. In practice, few (if any) tools actually support this. If that's the case, it may be less confusing to remove it from the spec.The text was updated successfully, but these errors were encountered: