Skip to content

Voeg regel toe voor error structuur bij Bad Requests#310

Open
TimvdLippe wants to merge 1 commit intodevelopfrom
bad-request-errors
Open

Voeg regel toe voor error structuur bij Bad Requests#310
TimvdLippe wants to merge 1 commit intodevelopfrom
bad-request-errors

Conversation

@TimvdLippe
Copy link
Contributor

Hier was initieel geen consensus voor bij de behandeling van het hoofdstuk "Error Handling".

Hier was initieel geen consensus voor bij de behandeling van het
hoofdstuk "Error Handling".
@TimvdLippe TimvdLippe requested review from joostfarla and strijm March 5, 2026 10:02
@github-actions github-actions bot added Status: In bewerking Het voorstel is in bewerking bij de beheerorganisatie. Overleg: TO-API Te agenderen voor het Technisch Overleg API labels Mar 5, 2026
@github-actions
Copy link

github-actions bot commented Mar 5, 2026

@TimvdLippe
Copy link
Contributor Author

Deze wijziging is gedeeltelijk goedgekeurd op het TO API van 2026-03-03. Alle regels behalve /core/error-handling/bad-request zijn goedgekeurd. Voor deze laatste regel wordt de discussie verder gevoerd met inachtneming van de input van Kadaster over hun huidige omgang van error handling (CC @strijm). Hier zal er expliciet worden gekeken naar de huidige oplossing bij Haal Centraal en worden gekeken naar:

  1. Het mogelijk hernoemen van errors naar invalid-params
  2. Analyseren van impact van name vervangen door location
  3. Kijken hoe de gedetailleerde specificatie van error handling in .feature bestanden bij Kadaster te vatten is in de structuur

#251 (comment)

Het mogelijk hernoemen van errors naar invalid-params

Dit lijkt me niet wenselijk, omdat je een JSON property niet kunt beschouwen als een "param" (in tegenstelling tot query en header parameters). Hier zit dus een semantische mismatch. Om deze reden was gekozen voor de algemenere term errors, in lijn met de voorbeelden in de RFC en bijv ook in dit blog.

Analyseren van impact van name vervangen door location

Het gebruik van name impliceert een enkelvoudige naam van een top-level property (in het geval van een body value). Dat lijkt ook de manier waarop het in de genoemde API's wordt gebruikt. Je wil echter ook errors kunnen rapporteren voor dieper gelegen waardes (zoals nested object properties of array items). Vandaar dat er gekozen is voor de bredere term location die voor body values een locator van de value verwacht (JSON Pointer voor JSON, XPath voor XML). Voor parameters is de waarde van location enkel de naam.

@joostfarla in #251 (comment)

Mogelijkheden voor naamgeving van het JSON veld:

  1. "errors"
  2. "invalid-params"
  3. "invalid-input"

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

Labels

Overleg: TO-API Te agenderen voor het Technisch Overleg API Status: In bewerking Het voorstel is in bewerking bij de beheerorganisatie.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant