-
Notifications
You must be signed in to change notification settings - Fork 150
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
CSP3: Allow report batching. #229
Comments
That would actually be a rather sensible default, no? But I guess making this backwards compatible would be hard. |
maybe a new directive instead of report-uri 'batched'? I also don't want to get the DoS reports from the old browser. |
Sure. |
Except the JSON will still have it, but it's better, yes. |
shrug We'd already be changing the JSON to allow an array of reports. That would be an opportunity to make larger changes if warranted. |
\o/ |
Something like
report-uri 'batched' /endpoint
would allow us to gather up several reports, and send them in a single request, rather than DDoSing the endpoint:The text was updated successfully, but these errors were encountered: