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

Improve replication document error when user_ctx or auth headers are not parsed correctly #1306

Open
nickva opened this issue May 1, 2018 · 2 comments

Comments

@nickva
Copy link
Contributor

nickva commented May 1, 2018

Adding something like

"user_ctx": null

to the replication document results in the state failed which is correct but the error reason is: "{error,{case_clause,null}}" which is not the best.

Same goes for using a null value for a header in a source/target endpoint object.

@nickva
Copy link
Contributor Author

nickva commented May 1, 2018

If not an error with a specific field identifier and value being mentioned at least it would be nice to have something like "document parsing failed".

@mikerhodes
Copy link
Contributor

This is almost something which would benefit from a (the horror!) schema validation to pre-flight this and catch lots of errors in one go?

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

No branches or pull requests

2 participants