-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
When fetch is used keepalive is the default, and Chrome only allows a POST body <= 65536 bytes in that scenario #1464
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
Comments
If anyone needs a workaround: Raven.config('<DSN>', {
fetchParameters: {
keepalive: false
}
}); |
update: the 64kb limit is part of the |
Thanks for the details. I'll look into this. https://fetch.spec.whatwg.org/#http-network-or-cache-fetch point 8.5.
|
thanks so much @kamilogorek! ✨ |
What is the current behavior?
raven-js
haskeepalive: true
in itsfetchDefaults
. Because of this, POSTs to Sentry usingfetch
will fail with a generic error if they are larger than 65536 bytes. This is smaller than Sentry's 100,000 byte limit (as I understand it). It's also smaller than theonpremise
limits (which are set by uWSGI, as I understand it).I opened this Chrome bug which helps explain further: https://bugs.chromium.org/p/chromium/issues/detail?id=871115&can=4&q=&colspec=ID%20Pri%20M%20Stars%20ReleaseBlock%20Component%20Status%20Owner%20Summary%20OS%20ModifiedWhat is the expected behavior?
I expect raven-js to be able to POST messages >= 65536 without failing.
Secondarily, it would be nice if there was some logging for failed log messages (basically echoing the sentiments of the people in the 413/too large issue comments)!
Versions
raven-js: 3.24.2
macOS Chrome: 68.0.3440.84
The text was updated successfully, but these errors were encountered: