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

Option validation #2

Open
wfarr opened this issue Apr 14, 2013 · 0 comments
Open

Option validation #2

wfarr opened this issue Apr 14, 2013 · 0 comments

Comments

@wfarr
Copy link
Owner

wfarr commented Apr 14, 2013

So I had a bit of a thought earlier. elasticsearch has a billion and one options for API requests, and it can be easy to typo one and shoot one's self in the foot without noticing as elasticsearch won't really complain.

It would involve a bit of maintenance overhead keeping up with elasticsearch releases, but for the most part the API is pretty stable.

We could well validate options on every endpoint and make super fucking useful error messages like: "indxees is not a valid option this endpoint. Must be one of [ :indexes ]" or whatever

Thoughts?

/cc @TwP @jbarnette

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant