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

Current Template API is not RESTful #3434

Closed
spinscale opened this issue Aug 2, 2013 · 0 comments · Fixed by #3436
Closed

Current Template API is not RESTful #3434

spinscale opened this issue Aug 2, 2013 · 0 comments · Fixed by #3436
Assignees
@spinscale
Labels
>enhancement v0.90.3 v1.0.0.Beta1

Comments

@spinscale
Copy link
Contributor

The current template API does not behave RESTful

  • HEAD on a template does not work
  • GET on a non-existing template returns a 200 status message
@ghost ghost assigned spinscale Aug 2, 2013
@spinscale spinscale mentioned this issue Aug 4, 2013
Merged
spinscale added a commit that referenced this issue Aug 5, 2013
@spinscale
Added index templates REST support for HEAD and proper 404
9f01260 
* Added HEAD support for index templates to find out of they exist
* Returning a 404 instead of a 200 if a GET hits on a non-existing index template

Closes #3434
mute pushed a commit to mute/elasticsearch that referenced this issue Jul 29, 2015
@spinscale
Added index templates REST support for HEAD and proper 404
757f97a 
* Added HEAD support for index templates to find out of they exist
* Returning a 404 instead of a 200 if a GET hits on a non-existing index template

Closes elastic#3434
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@spinscale spinscale

Labels
>enhancement v0.90.3 v1.0.0.Beta1
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Added index templates REST support for HEAD and proper 404 spinscale/elasticsearch
1 participant
@spinscale