ManifestStaticFilesStorage doesn't support trailing whitespace for JS source map references
#32383 added support for JS source map references, but the regex is incomplete. Browsers ignore trailing whitespace in source map references, but the regex we added did not allow for that. Whilst trailing whitespace is unlikely to be used, since source maps are normally generated by automated tools that shouldn't add it, we should ignore it to match browser behaviour.
Spotted during discussion on this PR for a similar fix on CSS source map references.
(Also as discussed, I tested the inital space being a tab, or more than one space, and those options are not supported by Firefox.)
Change History
(4)
Description: |
modified (diff)
|
Has patch: |
set
|
Owner: |
changed from nobody to Adam Johnson
|
Severity: |
Normal → Release blocker
|
Status: |
new → assigned
|
Triage Stage: |
Unreviewed → Accepted
|
Resolution: |
→ invalid
|
Severity: |
Release blocker → Normal
|
Status: |
assigned → closed
|
Triage Stage: |
Accepted → Unreviewed
|
It works without any changes.