-
Notifications
You must be signed in to change notification settings - Fork 9.1k
Proposal: Support for LINK and UNLINK HTTP methods #480
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
Comments
Here is the real RFC (HTTP 1.1), not a draft: And the IANA registry of all HTTP methods: |
RFC2616 which obsoletes RFC2068 says about PATCH/LINK/UNLINK:
But this RFC is dated of year 1999. Since, REST APIs and are born and PATCH/LINK/UNLINK become really useful. As Swagger already has support for PATCH, I don't see a reason to not support LINK/UNLINK too. |
nodejs' http-parser doesn't seem to support LINK/UNLINK... yet. |
The latest draft (as the name implies this is work in progress) about LINK/UNLINK: https://tools.ietf.org/html/draft-snell-link-method-12 |
Thx to @dolmen, nodejs http parser supports LINK & UNLINK When do you think the support of these HTTP methods can be available in swagger? |
They won't be added to the current swagger-spec (that wouldn't make any sense) but they can be considered for the next one. |
May I ask, how often do you create a new swagger spec? Thx |
historically, roughly every year we have a new spec. Recall all tooling needs to be updated with each change to the spec. |
I would like to vote htis one up, but I don't see any use in limiting this to link/unlink. As long as the browser can send it, it should be documentable imho. |
At showpad we use these in combination with a Link: header. |
Closing as a duplicate/subset of #1747 |
Hi,
I working on an app which has a REST interface where I use LINK and UNLINK HTTP methods to send messages. I would like to use swagger for documentation purposes, but unfortunately it does not support these two method types.
Have you considered the support of these?
RFC: http://tools.ietf.org/id/draft-snell-link-method-01.html
Regards,
Grunci
The text was updated successfully, but these errors were encountered: