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

Update csp_whitelist.xml #41

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Conversation

norgeindian
Copy link

This should fix the following issues in Magento 2 Checkout in version 2.4.7 or higher, where Magento_Csp module is active by default:

Refused to load the script 'https://api.lyra.com/static/js/krypton-client/V4.0/stable/kr-payment-form.min.js' because it violates the following Content Security Policy directive: "script-src ...
Refused to load the stylesheet 'https://api.lyra.com/static/js/krypton-client/V4.0/ext/classic-reset.css' because it violates the following Content Security Policy directive: "style-src

@diasnath
Copy link

Hello @norgeindian ,
Did you change the URL for Javascript client in your magento configuration ? In General Configuration > REST API KEYS section.

@norgeindian
Copy link
Author

@diasnath , don't know if I changed that some time ago, but it is set to https://api.lyra.com/static/

@norgeindian
Copy link
Author

@diasnath , can you reproduce this issue on your end?

@Ryldd
Copy link

Ryldd commented Sep 2, 2024

Hello @norgeindian,
the url should not be https://api.lyra.com/static/ but https://static.lyra.com/static/
Can you try with it ?

@norgeindian
Copy link
Author

@Ryldd , I tried to change that, but as soon as I save the config at this point, I get the error message Invalid file name.
That happens even when I do not change anything.
Debugged that a bit and found out, that you define some default file names for logos. In my case, Magento looks for them in pub/media/lyra/images, although they are not present there and only in pub/static folder.
Have you already heard of this issue? How can this be fixed?

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.

3 participants