Skip to content

Adding content-security-policy to webview content#95

Merged
jalissia merged 4 commits intomasterfrom
user/jalissia/content-security-policy
Sep 3, 2019
Merged

Adding content-security-policy to webview content#95
jalissia merged 4 commits intomasterfrom
user/jalissia/content-security-policy

Conversation

@jalissia
Copy link
Member

This PR fixes #91 by adding a content-security-policy meta tag to the webview html and the devtools.html iframe.

  • Split out the inline css into a separate file so that we don't need to use styles-src 'unsafe-inline'
  • Update build steps to copy new css file to out directory
  • Add new content-security-policy to lock down the webview content to just files we provide
  • Added missing body tag to webview content

@jalissia jalissia added the bug Something isn't working label Aug 23, 2019
Copy link

@mjbvz mjbvz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the quick turn around!

@jalissia jalissia merged commit 73e37ef into master Sep 3, 2019
@jalissia jalissia deleted the user/jalissia/content-security-policy branch September 3, 2019 19:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Webview does not set a content security policy

3 participants