Skip to content

[nlgov:problem-invalid-input] validatie in ADR 2.2.0 is te strict #368

Open

Description

@sindlar-kadaster

De validatieregel voor [nlgov:problem-invalid-input] rapporteert voor GET operaties met path-parameters die geen 400 Bad Request response retourneren een validatiefout met de melding [nlgov:problem-invalid-input] GET and DELETE endpoints that have parameters, and all other endpoints must be able to return a 400 response.

Dat is naar mijn mening te strict want het veronderstelt dat de server altijd de path-parameter valideert en criteria hanteert om deze af te wijzen, terwijl het in geval van een path-parameter prima denkbaar is dat de server alles accepteert wat binnenkomt en simpelweg probeert de resource te vinden, resulterend in een 404 Not Found als dat niet lukt. En dit punt kan denk ik nog breder getrokken worden naar GET parameters in het algemeen. Mogelijk is de validatieregel een te stricte interpretatie van de gedachte dat als de API parameters valideert, eventuele fouten dan dienen te worden gecommuniceerd d.m.v. een 400 Bad Request response?

Om dit te reproduceren, plaats in het DON Checker voorbeeld de parameters onder /api/{id} een niveau lager, onder de get:

    "/api/{id}": {
      "get": {
        "parameters": [
          {
            "$ref": "#/components/parameters/id"
          }
        ],
        "operationId": "retrieveApi",

Zie ook issue #367 .

Activity

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions