Voeg regel toe over het documenteren van server URLs #294
Voeg regel toe over het documenteren van server URLs
#294TimvdLippe wants to merge 3 commits intodevelopfrom
Conversation
Hiermee maken we een eerste stap in het enforceren van informatie over servers. Tevens maken we hiermee duidelijk dat een relatieve URL is toegestaan, echter moet er altijd minstens 1 absolute URL zijn.
| <dd> | ||
| OpenAPI definition document MUST include at least one Server object in <a href="https://spec.openapis.org/oas/v3.0.1.html#server-object"><code>servers</code></a>. | ||
| One or more Server objects MUST have a <code>url</code> which is an absolute URL. | ||
| At most one Server object MAY have a <url>url</code> which is a relative URL. |
There was a problem hiding this comment.
Is deze uitzondering echt nodig? Zou het niet beter zijn om altijd absolute URLs te vereisen?
There was a problem hiding this comment.
Dat is het voorstel in #290, zie dat issue voor meer context.
|
Dit blijft een ingewikkelde discussie. Als je ervan uitgaat dat een OAS op prod:
test:
Wat ik heel wenselijk zou vinden zou de classificatie van een omgeving zijn, zodat je weet welke server bij prod, test, dev, etc. hoort (ook tbv het API register). Maar die optie heeft OAS niet, dus dan zouden we iets af moeten spreken over de volgorde, naam of een extensie, zoals beschreven in #208. Mogelijk maakt het deze discussie makkelijker als we daar iets voor verzonnen hebben? |
|
Ik heb Spectral regels kunnen schrijven en gecheckt of er API's wel/niet aan voldoen. Met deze regels (dus max 1 relative URL), voldoet er 1 API niet: het API register van developer.overheid.nl (waarschijnlijk vergeten te updaten?) Als we helemaal geen relatieve URLs toestaan, dan zijn dit de statistieken: Voorbeelden: Het lijken vooral DSO API's te zijn die dit patroon volgen |
Hiermee maken we een eerste stap in het enforceren
van informatie over servers. Tevens maken we hiermee
duidelijk dat een relatieve URL is toegestaan, echter
moet er altijd minstens 1 absolute URL zijn.
Part of #208
Fixes #290