Skip to content
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

Attempt to fix CSP to allow RunLLM widget #767

Merged
merged 1 commit into from
Jun 18, 2024

Conversation

cw75
Copy link
Contributor

@cw75 cw75 commented Jun 14, 2024

⚠️   Pre Checklist

Please complete ALL items in this checklist, and remove before submitting

  • I have npm run build and npm run serve locally before submitting this PR
  • I have read through the Contributing Documentation

Summary

This PR attempts to fix the CSP issue that's blocking the RunLLM widget from loading. Previously we weren't allowing the parent url https://www.google.com/, which may be the issue.

Does this close any open issues?

Closes xx

Screenshots

Include any relevant screenshots here.

Other Information

Any other information that is important to this PR.

@cw75 cw75 mentioned this pull request Jun 17, 2024
2 tasks
@klesh klesh merged commit 65290ea into apache:main Jun 18, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants