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

Reject delete index requests with a body #37501

Closed

Conversation

Projects
None yet
4 participants
@jasontedor
Copy link
Member

commented Jan 15, 2019

Delete index requests do not take a body, and it is misleading and arguably a bug that they currently do. We abhor this sort of leniency in Elasticsearch, so this commit removes the possibility for a delete index request to have a body.

Relates #8217

jasontedor added some commits Jan 15, 2019

Reject delete index requests with a body
Delete index requests do not take a body, and it is misleading and
arguably a bug that they currently do. We abhor this sort of leniency in
Elasticsearch, so this commit removes the possibility for a delete index
request to have a body.
@elasticmachine

This comment has been minimized.

Copy link

commented Jan 15, 2019

jasontedor added some commits Jan 15, 2019

@dakrone
Copy link
Member

left a comment

LGTM

@jasontedor

This comment has been minimized.

Copy link
Member Author

commented Jan 15, 2019

Superseded by #37504

@jasontedor jasontedor closed this Jan 15, 2019

@jasontedor jasontedor deleted the jasontedor:delete-index-with-a-body branch Jan 15, 2019

@jasontedor

This comment has been minimized.

Copy link
Member Author

commented Jan 15, 2019

I decided to solve this once and for all and opened #37504.

@colings86 colings86 added v7.0.0-beta1 and removed v7.0.0 labels Feb 7, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.