search

belgif / rest-guide

REST Guidelines of Belgian government institutions
https://www.belgif.be/specification/rest/api-guide/
Apache License 2.0
24 stars 4 forks source link

rename unknownParameter issue type #140

Closed pvdbosch closed 9 months ago

pvdbosch commented 11 months ago

A body or body part isn't considered a parameter (anymore) in OpenAPI 3. Hence we ought to rename urn:problem-type:belgif:input-validation:unknownParameter into something else

jpraet commented 11 months ago

urn:problem-type:belgif:input-validation:unknownInput?

pvdbosch commented 11 months ago

WG decision: can be updated bc initial release of the rule was recent so very little chance on breaking something Also clarify in rule that it applies to json properties in the request body, and doesn't apply to path parameters ( bc they don't have a name on the wire, and would result in a 404 instead)

pvdbosch commented 10 months ago

updated by #144. Has already been merged without PR review to limit risk on impact of change of input-validation issue type.

Keeping issue open for review next WG.

pvdbosch commented 9 months ago

Processed remarks on #144 in new (merged) PR #154