VNG-Realisatie / gemma-zaken

Samen ontwikkelen van API's voor Zaakgericht werken
https://vng-realisatie.github.io/gemma-zaken/
Other
41 stars 27 forks source link

DRC referentie implementatie gebruikt niet de juiste versie #2292

Open johannesbattjes opened 1 year ago

johannesbattjes commented 1 year ago

Bug

Als we 1.0 mee geven als Api-version krijgen we toch 1.3 terug versioning of VNG reference

joeribekker commented 1 year ago

Hoewel ik niet van VNG ben kan ik al wel aangeven dat dit nooit is ingebouwd in de referentie implementaties.

Heel eerlijk vind ik de meerwaarde ook zeer beperkt omdat minor versies backwards compatible (zouden) moeten zijn. Major versies maken het onderscheid al in de URL. Ben wel benieuwd naar jullie use case hiervoor.

johannesbattjes commented 1 year ago

@joeribekker de use case is dat wij met wat testjes wilden checken welk gedrag correct is met betrekking tot het veld inhoud en bestandsomvang, waar we al een anderhalf jaar lang veel problemen mee hebben bij diverse testtrajecten. Ik begrijp ook niet wat de relevantie is van de "meerwaarde". Het gaat er toch om wat de standaard zegt? Als de referentie-applicatie al niet de standaard hoeft te volgen omdat iemand eerst oordeelt dat sommige delen van de standaard niet relevant zijn, welke applicatie dan wel? Wij hebben deze check op versie wel ingebouwd in fundament (onze DRC). Wil je wellicht een voorstel doen deze regel te laten vervallen in de standaard? Dat zou ons dan werk schelen bij het ondersteunen van nieuwere versies. In dat geval zal ik rondvragen of wij daar problemen mee zien.

michielverhoef commented 1 year ago

Het laten vervallen van de verplichting om in een lagere versie antwoord te kunnen krijgen maakt volgens mij elke wijziging aan de standaard een major wijziging die niet backwards compatible is. Als consumer die een lagere versie gebruikt krijg je immers een response met mogelijk meer/andere velden of waarden dan waar je mee om kunt gaan.

Het lijkt me dus onverstandig deze verplichting te laten vervallen.

Dan blijft over het gedrag van de referentie implementatie: Dat is inderdaad niet conform de standaard. Indertijd is hier bewust voor gekozen omdat we op dat moment de middelen niet hadden deze regel in te bouwen. Aan die situatie is helaas niets veranderd, in tegendeel zou ik bijna willen zeggen. Het is dus zeker een bug, alleen kan ik op dit moment geen toezegging doen dat dit op korte termijn opgepakt gaat worden.