-
Notifications
You must be signed in to change notification settings - Fork 183
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
Please update CSP with new avatar URL #817
Comments
@clouserw I'm having issues recreating this issue. Here's where we set the CSP policy for the profile images. We're using @tcinotto Can you provide more details on where you were getting this error? I may need to work with you directly to debug/reproduce this. |
It looks like @groovecoder fixed this in abf32ec If that commit is already live, maybe there is some caching going on? I can still reproduce as of this morning. Steps:
|
@clouserw Thanks for the follow up – I had a profile image. Confirmed. I would imagine the caching would no longer be an issue, so I'll keep digging. |
Fix #817 - Remove map function, set URLs directly
Reported on Slack:
Relay is blocking the avatar URL from FxA:
Resulting in mis-rendering of the account menu in the top left.
The fix is to add
profile.accounts.firefox.com
to the img-src CSP ruleThe text was updated successfully, but these errors were encountered: