-
Notifications
You must be signed in to change notification settings - Fork 570
Making syscall/js CSP compatible #1168
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
Making syscall/js CSP compatible #1168
Conversation
Tested locally with a simple client and server that does CSP.
Throws an error like I expect on older versions of gopherjs. No error with these updates. |
CI tests don't test for CSP compliance (obviously 🙃), but they do run the standard |
Have to log off for the evening and then I'm out of the office tomorrow, but will debug the failures Wednesday morning CDT. |
Thank you for your contribution! |
Making the
syscall/js
package compatible with a CSP that does not allowscript-src 'unsafe-eval'
. See this thread for background.cc @nevkontakte can you merge this if you think it's good? I tested a CSP manually and it worked, put some snippets below.