-
Notifications
You must be signed in to change notification settings - Fork 10
Voeg meer http methods toe aan core/http-methods #223
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: develop
Are you sure you want to change the base?
Conversation
Hiermee maken we het expliciet dat ook de HTTP methods die enkel (metadata) informatie ophalen toegestaan zijn. Deze methoden maken geen wijzigingen of lezen resources uit, maar kunnen gebruikt worden om informatie over het endpoint zelf uit te lezen.
De regel is geen white list van toegestane methods. Enkel onjuist gebruik van methods is niet toegestaan (lastig te testen met enkel openapi.json). Als we toch overgaan naar een white list lijkt |
Op basis van ons gesprek heb ik de lijst nu optioneel gemaakt en non-exhaustive. Daarmee laten we het open, maar geven we ook expliciet de meest gebruikte methodes aan. Tevens heb ik wat informatie die in de "note" stond in het statement gezet, om zo de normatieve regels en de verduidelijking te scheiden van elkaar. |
…onal-http-methods
Actiepunt van TO: @sanderke HTTP RFC opnieuw lezen en kijken wat de intentie is. Tevens inlezen op OpenAPI 3.2 waar meer methoden worden toegevoegd (OAI/OpenAPI-Specification#1747) |
<div class="note"> | ||
<p>The HTTP specification [[rfc9110]] offers a set of standard methods, where every method is designed with explicit semantics. | ||
The OpenAPI Specification 3.0 <a href="https://spec.openapis.org/oas/v3.0.1#path-item-object">Path Item Object</a> also supports these methods, except for <code>CONNECT</code>. | ||
<p>In addition to the standard HTTP methods, a server may support other optional methods as well, e.g. <code>PROPFIND</code>, <code>COPY</code>, <code>PURGE</code>, <code>VIEW</code>, <code>LINK</code>, <code>UNLINK</code>, <code>LOCK</code>, <code>UNLOCK</code>, etc. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
<p>In addition to the standard HTTP methods, a server may support other optional methods as well, e.g. <code>PROPFIND</code>, <code>COPY</code>, <code>PURGE</code>, <code>VIEW</code>, <code>LINK</code>, <code>UNLINK</code>, <code>LOCK</code>, <code>UNLOCK</code>, etc. | |
<p>In addition to the standard HTTP methods, a server may support other optional methods as well, e.g. <code>PROPFIND</code>, <code>COPY</code>, <code>PURGE</code>, <code>VIEW</code>, <code>LINK</code>, <code>UNLINK</code>, <code>LOCK</code>, <code>UNLOCK</code>, etc. Additional methods MAY be documented using OpenAPI 3.2 while preserving these semantics defined in [rfc9110]. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ik weet niet of we hier al kunnen verwijzen naar 3.2. Misschien kunnen we dit in een niet-normatieve noot toevoegen van "dit komt er in de toekomst aan"?
Co-authored-by: Alexander Green <[email protected]>
Hiermee maken we het expliciet dat ook de HTTP methods die enkel (metadata) informatie ophalen toegestaan zijn. Deze methoden maken geen wijzigingen of lezen resources uit, maar kunnen gebruikt worden om informatie over het endpoint zelf uit te lezen.
Fixes Geonovum/KP-APIs#633