CDN Example: Add integrity metadata map and SRI hashes + Github Action to update#4100
CDN Example: Add integrity metadata map and SRI hashes + Github Action to update#4100magicmark wants to merge 8 commits intographql:mainfrom
Conversation
|
|
Holy crap thank you, I was so frustrated trying to put this together myself when trying to integrate GraphiQL into my app where I use SRI for everything else. I hope this gets merged. |
jonathanawesome
left a comment
There was a problem hiding this comment.
LGTM, thanks @magicmark!
trevor-scheer
left a comment
There was a problem hiding this comment.
This is excellent, thanks! Once I've got permissions sorted on this repo I'll get this merged. I literally just opened a related issue.
This reference implementation will likely be copied and pasted in a bunch of places.
SRI hashes are important when loading from CDNs, so hopefully this improves security in a bunch of places :)
(I've verified manually this file still works)
I've also set up a github action to regenerate the example when:
releasedaction for updated internal deps