Navigation Menu

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

Jump to bottom

Clear scroll should accept scroll_id in body #5726

Closed
clintongormley opened this issue Apr 8, 2014 · 0 comments
Closed

Clear scroll should accept scroll_id in body #5726

clintongormley opened this issue Apr 8, 2014 · 0 comments
Assignees
@martijnvg
Labels
>bug v1.0.3 v1.1.1 v1.2.0 v2.0.0-beta1

Comments

@clintongormley
Copy link

In elastic/elasticsearch-perl#24 a user was generating scroll IDs which were too long for the intervening proxy to handle in the URL or query string. The same problem would apply to the clear-scroll API.

The clear-scroll API should optionally accept the scroll_id in the body as well as in the URL
@martijnvg martijnvg self-assigned this Apr 8, 2014
martijnvg added a commit to martijnvg/elasticsearch that referenced this issue Apr 8, 2014
@martijnvg
The clear scroll apis now optionally accepts a scroll_id in it body.
d1a0604 
Closes elastic#5726
@martijnvg martijnvg mentioned this issue Apr 8, 2014
Closed
@Mpdreamz Mpdreamz mentioned this issue Apr 8, 2014
Closed
martijnvg added a commit that referenced this issue Apr 11, 2014
@martijnvg
The clear scroll apis now optionally accepts a scroll_id in it body.
d0e6ef2 
Closes #5726
martijnvg added a commit that referenced this issue Apr 11, 2014
@martijnvg
The clear scroll apis now optionally accepts a scroll_id in it body.
78a9781 
Closes #5726
martijnvg added a commit that referenced this issue Apr 11, 2014
@martijnvg
The clear scroll apis now optionally accepts a scroll_id in it body.
25eed49 
Closes #5726
mute pushed a commit to mute/elasticsearch that referenced this issue Jul 29, 2015
@martijnvg
The clear scroll apis now optionally accepts a scroll_id in it body.
e61d648 
Closes elastic#5726
mute pushed a commit to mute/elasticsearch that referenced this issue Jul 29, 2015
@martijnvg
The clear scroll apis now optionally accepts a scroll_id in it body.
a14510e 
Closes elastic#5726
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@martijnvg martijnvg

Labels
>bug v1.0.3 v1.1.1 v1.2.0 v2.0.0-beta1
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@clintongormley @martijnvg