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

Best effort attempt to transmit the error #2

Closed
igrigorik opened this issue Oct 13, 2014 · 2 comments
Closed

Best effort attempt to transmit the error #2

igrigorik opened this issue Oct 13, 2014 · 2 comments

Comments

@igrigorik
Copy link
Member

From: http://lists.w3.org/Archives/Public/public-web-perf/2014Jul/0095.html

ttuttle@: I'd like to allow the user-agent to retry the uploads if they fail. If the issue is a transient network issue (i.e. a route is flapping), it's a waste to throw out the error report just because the network was still glitched the first time the upload was attempted.

aheady@: This reads like a denial of service attack. We did discuss it originally, but how do you control the retries when an origin has a short lived but widespread spike in errors, especially when the origin for the error is also the origin/logging endpoint for these navigation error calls. A few seconds after it recovers it gets hit with a global surge in telemetry request, knocking if offline, more errors…... Also goes back to #1, any error that is stable enough to repro is going to be reported by a large number of users. I expect this system to be lossy telemetry wise. Optimized to protect the origin, not the error telemetry. And if you wait for the next successful page load, then you can get the errors from the queue.

Mirroring the language in Beacon, I think we can (read, should) defer this decision to the UA:
"The User Agent SHOULD make a best effort attempt to eventually transmit the data."

Above reserves the right to retry, but it does not imply that it must be implemented.

@igrigorik
Copy link
Member Author

  • consider starting with single report URL, but have some provisions for multiple report URLs?

@igrigorik
Copy link
Member Author

Resolved in new draft, closing.

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

No branches or pull requests

1 participant