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

Process graph validation returns only a single error #144

Closed
m-mohr opened this issue Oct 31, 2018 · 1 comment
Closed

Process graph validation returns only a single error #144

m-mohr opened this issue Oct 31, 2018 · 1 comment
Labels
Milestone

Comments

@m-mohr
Copy link
Member

m-mohr commented Oct 31, 2018

Currently, the validation of process graphs (POST /validation) can only return a single error and a user would need to request a list of errors for individual error. It would be much more user-friendly if the user could get a list of all errors in a single request.

List of errors could be returned as follows:

{
  "errors":[
    {
      "id":"1541025804936",
      "code":2103,
      "message":"The value specified for the process argument 'extent' in process 'filter_bbox' is invalid: instance.north is not of a type(s) number",
      "url":"https://open-eo.github.io/openeo-api/errors/#openeo-error-codes"
    },
    {
      "id":"1541026583276",
      "code":2103,
      "message":"The value specified for the process argument 'extent' in process 'filter_bbox' is invalid: instance additionalProperty \"base\" exists in instance when not allowed",
      "url":"https://open-eo.github.io/openeo-api/errors/#openeo-error-codes"
    }
  ],
  "links":[
  ]
}
@m-mohr m-mohr added the other label Oct 31, 2018
@m-mohr m-mohr added this to the v0.4 milestone Oct 31, 2018
@m-mohr
Copy link
Member Author

m-mohr commented Nov 7, 2018

Process graph validation (POST /validation) now returns a 200 HTTP status code for both valid and invalid process graphs and responds with a list of errors. Validation errors are still a valid response for a validation request and thus should respond with a success status code such as 200.

@m-mohr m-mohr closed this as completed Nov 7, 2018
m-mohr added a commit that referenced this issue Nov 7, 2018
…200 for valid and invalid process graphs and responds with a list of errors. #144
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