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

CSP add report-uri #33

Closed
vasco-santos opened this issue Dec 15, 2022 · 1 comment · Fixed by nftstorage/nftstorage.link#209 or #39
Closed

CSP add report-uri #33

vasco-santos opened this issue Dec 15, 2022 · 1 comment · Fixed by nftstorage/nftstorage.link#209 or #39
Assignees
Labels
kind/enhancement A net-new feature or improvement to an existing feature need/triage Needs initial labeling and prioritization

Comments

@vasco-santos
Copy link
Member

We should have an API where we can receive reports from URIs being blocked by CSP, so that we can have a picture of how this affects users, as well as how we can make it better.

https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Content-Security-Policy#reporting_directives

report-uri Instructs the user agent to report attempts to violate the Content Security Policy. These violation reports consist of JSON documents sent via an HTTP POST request to the specified URI

cc @jchris

@vasco-santos vasco-santos added kind/enhancement A net-new feature or improvement to an existing feature need/triage Needs initial labeling and prioritization labels Dec 15, 2022
@vasco-santos vasco-santos self-assigned this Dec 15, 2022
@jchris
Copy link

jchris commented Dec 15, 2022

@heyjay44 this is related to a bug report / fix for hosting HTML apps on our gateway.

  • timely (capturing a snapshot of our usage as people were doing before a recent policy change, so if we wait too long those errors will taper off)
  • lightweight (I think only a few lines of code plus test and release cycle)
  • independent (could be shipped with or without other changes going on)

I'm not saying we should change the schedule for it but if it sneaks in, that's why :)

PS here is a screenshot of how our policy change can impact apps that were running previously:

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/enhancement A net-new feature or improvement to an existing feature need/triage Needs initial labeling and prioritization
Projects
None yet
2 participants