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

CSP3: Allow report batching. #229

Open
mikewest opened this issue Mar 26, 2015 · 6 comments
Open

CSP3: Allow report batching. #229

mikewest opened this issue Mar 26, 2015 · 6 comments
Labels
Milestone

Comments

@mikewest
Copy link
Member

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:

[
    {
        // report 1
    },
    {
        // report 2
    },
    ...
    {
        // report N
    }
]
@mikewest mikewest added the CSP label Mar 26, 2015
@mikewest mikewest added this to the CSP 3 milestone Mar 26, 2015
@annevk
Copy link
Member

annevk commented Mar 26, 2015

That would actually be a rather sensible default, no? But I guess making this backwards compatible would be hard.

@devd
Copy link
Contributor

devd commented Mar 27, 2015

maybe a new directive instead of report-uri 'batched'? I also don't want to get the DoS reports from the old browser.

@mikewest
Copy link
Member Author

Sure. reports /endpoint? Then we can drop uri as well and make @annevk happy.

@annevk
Copy link
Member

annevk commented Mar 27, 2015

Except the JSON will still have it, but it's better, yes.

@mikewest
Copy link
Member Author

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.

@annevk
Copy link
Member

annevk commented Mar 27, 2015

\o/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants