Skip to content

Use webview CSP source for style-src#1170

Merged
charisk merged 2 commits intomainfrom
charisk/src-style-csp-fix
Feb 25, 2022
Merged

Use webview CSP source for style-src#1170
charisk merged 2 commits intomainfrom
charisk/src-style-csp-fix

Conversation

@charisk
Copy link
Copy Markdown
Contributor

@charisk charisk commented Feb 25, 2022

Minor change to use webview.cspSource instead of a hard-coded value.

Checklist

N/A

  • CHANGELOG.md has been updated to incorporate all user visible changes made by this pull request.
  • Issues have been created for any UI or other user-facing changes made by this pull request.
  • [Maintainers only] If this pull request makes user-facing changes that require documentation changes, open a corresponding docs pull request in the github/codeql repo and add the ready-for-doc-review label there.

@charisk charisk requested a review from a team as a code owner February 25, 2022 11:05
@aeisenberg
Copy link
Copy Markdown
Contributor

aeisenberg commented Feb 25, 2022

Oops. I see you included extensions/ql-vscode/.DS_Store in the commit. Can you remove that?

@charisk
Copy link
Copy Markdown
Contributor Author

charisk commented Feb 25, 2022

Oops. I see you included extensions/ql-vscode/.DS_Store in the commit. Can you remove that?

How embarrassing! Deleted.

@charisk charisk merged commit 735f177 into main Feb 25, 2022
@charisk charisk deleted the charisk/src-style-csp-fix branch February 25, 2022 16:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@aeisenberg aeisenberg aeisenberg approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@charisk @aeisenberg