-
Notifications
You must be signed in to change notification settings - Fork 8.1k
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
[Reporting] fix too_long_frame_exception by passing scroll_id in the request body #102972
[Reporting] fix too_long_frame_exception by passing scroll_id in the request body #102972
Conversation
b21ed81
to
618a760
Compare
@elasticmachine merge upstream |
fd8f58c
to
7a35363
Compare
Pinging @elastic/kibana-app-services (Team:AppServices) |
Pinging @elastic/kibana-reporting-services (Team:Reporting Services) |
@elasticmachine merge upstream Sorry @tsullivan, the cloud instance behind ci-stats had a little fit so that build wasn't going to succeed due to metrics at least, so I started it over. |
💚 Build SucceededMetrics [docs]
History
To update your PR or re-run it, just comment with: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great job finding this @tsullivan ! I had no idea that scroll ids were that big potentially.
…request body (elastic#102972) * pass scroll_id in the request body not a param * update test to match Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
…request body (elastic#102972) * pass scroll_id in the request body not a param * update test to match Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
…request body (elastic#102972) * pass scroll_id in the request body not a param * update test to match Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
…request body (elastic#102972) * pass scroll_id in the request body not a param * update test to match Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
Summary
Fixes #102971
An HTTP line can not be longer than 4096 bytes. This becomes a problem when using scan and scroll with Elasticsearch, and having the scroll ID placed in the path of the URL.
Bad:
When the URL is too long, Elasticsearch may respond with an error:
The scroll ID should be in the POST payload body.
Good:
Checklist
Delete any items that are not applicable to this PR.
Release note
Fixed an issue with CSV export that could limit the size of the export to 500 lines.