-
Notifications
You must be signed in to change notification settings - Fork 9k
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
Add UI support for SEARCH HTTP verb/method #10085
Comments
Would Like to work on this issue.. |
#10086 🤷 |
OpenAPI doesn't seem to support the SEARCH verb as part of the spec: https://spec.openapis.org/oas/latest.html#path-item-object |
OpenAPI was last published on February 15, 2021, and is outdated due to daily technical changes. I use NestJS, which supports the Search method, but Swagger UI does not support it. |
@tuan-nguyen-wareflex You can help change the OpenAPI specification, there is a discussion here: OAI/OpenAPI-Specification#1747 related to different HTTP verbs and whether they have massive support in the wild since SEARCH comes from WebDAV which is/was a Microsoft specific technology/RFC |
@JaredAAT, please note that though the specs may not support is, OpenAPI actually DOES support it EXCEPT for the UI. The proposed fix is to align the UI with the generated definitions so it displays what the OpenAPI definitions already support |
@doronguttman well no... lots of confusion in that sentence there. OpenAPI does not support it, you may think that because you can add any verb to the specs that it does, but in actuality, until the issue i highlighted in my previous comment, OpenAPI does not support SEARCH or various other verbs that are not listed in the specs here: https://spec.openapis.org/oas/latest.html#path-item-object There is no UI to OpenAPI. Swagger (which is what we're commenting on here) is a UI that can display OpenAPI documents/specs but is not the UI of OpenAPI, there are other UI's that you can get that display OpenAPI specs. If swagger were to support SEARCH as a verb, they'd be operating outside of the OpenAPI specification |
Content & configuration
Swagger/OpenAPI definition:
Swagger-UI configuration options:
Is your feature request related to a problem?
Add UI support for SEARCH verb
Describe the solution you'd like
add the SEARCH verb to the list of supported methods. swagger itself already supports it
Describe alternatives you've considered
tried to see if there's a plugin available and did not find one
Additional context
we have an endpoint that is used to search for a record (does not mutate) and has multiple required arguments to be provided in the body of the request. GET with body is not supported by most frameworks. SARCH is becoming quite common
The text was updated successfully, but these errors were encountered: