openservicebrokerapi / servicebroker

Open Service Broker API Specification
https://openservicebrokerapi.org/
Apache License 2.0
1.19k stars 434 forks source link

Spec does not define expected status code when required fields are absent #274

Closed krancour closed 7 years ago

duglin commented 7 years ago

While I'm not sure we need to say when to use all possible http error codes, I think this is a good one to call out and I'm thinking that a "400 Bad Request" is appropriate.

krancour commented 7 years ago

@duglin and that is how I've implemented it in the broker I am currently working on (because that seems the most common sense response), but like you, I do believe the spec should be explicit in this regard.

duglin commented 7 years ago

PR: https://github.com/openservicebrokerapi/servicebroker/pull/275