Open Bug 1834808 Opened 1 year ago Updated 3 months ago

Local script override breaks SRI

Categories

(DevTools :: Debugger, defect, P3)

defect

Tracking

(Not tracked)

People

(Reporter: denschub, Unassigned)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

STR:

  1. Open this testcase, which is also attached.
  2. Observe that the body contains a "hello, world!" message
  3. Open the JS Debugger, search the main.js script in the sidebar, add a script override
  4. Save the file without any changes
  5. Reload

Expected:

The testcase should still work fine.

Actual:

JS fails. In the console, you can see

“data:application/x-javascript;base64,...” is not eligible for integrity checks since it’s neither CORS-enabled nor same-origin.

and even if the resource would be eligible for SRI checks, they'd fail as soon as you change the contents of that script file, so it would be best to turn off SRI checks at all for scripts with local overrides.

I'll note that I have yet experience this as an actual issue while doing diagnosis work, and Tom has only seen it once. I'm filing this primarily to not forget about this, but this probably has low priority.

I will, however, also note that cdnjs generates <script> tags that have both a CORS-attribute and an integrity attribute by default.

Severity: -- → S3
Priority: -- → P3

I came across this when trying to add an override for a js file for Snapchat web in bug1829758.

...
<script defer="defer" src="https://cf-st.sc-cdn.net/dw/e41b4fa927f08042aae1.js" integrity="sha384-NGo9RJSNo5HxnsjPeTdvDdpPf+7TonmTIuQZcktoEuxK9oLS48By63RPKDJPkiHN" crossorigin="anonymous"></script>
<script defer="defer" src="https://cf-st.sc-cdn.net/dw/7c79e428a9c4f60e632e.js" integrity="sha384-zHpeyVB9SD7JaH8OB1VYA9fxxMkyzo1Qg5CB3FdifC8SN26WE58Ja8O12SswFT9k" crossorigin="anonymous"></script>

A possible workaround is to add an override for the html file and remove integrity attribute, but that depends on bug1849920, I believe.

You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: