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

OPA returning 200 OK when input document not supplied #227

Closed
tsandall opened this issue Jan 24, 2017 · 0 comments
Closed

OPA returning 200 OK when input document not supplied #227

tsandall opened this issue Jan 24, 2017 · 0 comments
Labels

Comments

@tsandall
Copy link
Member

OPA is returning 200 OK when evaluating queries that depend on an input document even if an input document has not been supplied. Per the documentation, this should result in a 400 Bad Request response.

@tsandall tsandall added the bug label Jan 24, 2017
tsandall added a commit to tsandall/opa that referenced this issue Jan 24, 2017
These changes update OPA to analyze queries to determine if an input
document is required. In the future, more sophisticated checks could be
performed (e.g., JSON schema validation).

If an input document is required but not provided, OPA will return HTTP
400 (per the documentation). This was broken in open-policy-agent#197.

Fixes open-policy-agent#227
tsandall added a commit that referenced this issue Jan 25, 2017
These changes update OPA to analyze queries to determine if an input
document is required. In the future, more sophisticated checks could be
performed (e.g., JSON schema validation).

If an input document is required but not provided, OPA will return HTTP
400 (per the documentation). This was broken in #197.

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

No branches or pull requests

1 participant