-
Notifications
You must be signed in to change notification settings - Fork 362
KTOR-9165 and KTOR-9193 Documentation for OpenAPI specification generation #744
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: 3.4.0
Are you sure you want to change the base?
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
3c9da2d to
ef10e78
Compare
codeSnippets/snippets/openapi-spec-gen/src/main/kotlin/com/example/Application.kt
Outdated
Show resolved
Hide resolved
topics/openapi-spec-generation.md
Outdated
| #### Supported KDoc fields | ||
|
|
||
| | Tag | Format | Description | | ||
| |-----------------|-------------------------------------------------|----------------------------------| | ||
| | `@tag` | `@tag *name` | Groups the endpoint under a tag | | ||
| | `@path` | `@path [Type] name description` | Path parameter | | ||
| | `@query` | `@query [Type] name description` | Query parameter | | ||
| | `@header` | `@header [Type] name description` | Header parameter | | ||
| | `@cookie` | `@cookie [Type] name description` | Cookie parameter | | ||
| | `@body` | `@body contentType [Type] description` | Request body | | ||
| | `@response` | `@response code contentType [Type] description` | Response definition | | ||
| | `@deprecated` | `@deprecated reason` | Marks the endpoint as deprecated | | ||
| | `@description` | `@description text` | Extended description | | ||
| | `@security` | `@security scheme` | Security requirements | | ||
| | `@externalDocs` | `@external href` | External documentation link | |
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.
For 3.4.0, we're updating the comment API style from KDoc tags to use a custom markdown format, due to some concerns over the tags potentially causing some unexpected behaviour in the IDE and KDoc processing. I just changed this last week after some comments from @whyoleg, so it's pretty last-minute 😅
So now all these keywords instead just need to be at the start of the line with a colon divider to designate the value:
/**
* Get a list of widgets.
*
* - Tags:
* - widgets
* - admin
* - Cookie: X-Analytics-Session some tracking cookie
* - Responses:
* - 200 application/json [Widget]+ A list of widgets
* - 400 Invalid request
*/
Note, the plural form with tags and responses is an optional format, you can also use Tag: widgets, and the list bullets at the top level are also optional and just a matter of preference for formatting.
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.
Thanks for the tip. I've updated the section to reflect the changes.
topics/openapi-spec-generation.md
Outdated
| To expose the OpenAPI specification through an interactive UI, use the [OpenAPI](server-openapi.md) | ||
| and [SwaggerUI](server-swagger-ui.md) plugins. |
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.
I'm not sure if we have good documentation for the difference between the two plugins. I don't really know the full extent of the differences, but from what I can tell the main difference is that the OpenAPI renders the HTML from the provided model on the back end, and SwaggerUI renders the details from the front end and exposes the model endpoint as JSON or YAML. I would say the Swagger plugin ought to be the default implementation since it's a more modern UI and it's easier to navigate. Maybe @e5l could have some thoughts on this.
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.
Added a couple of bullet points to clarify the difference between the two and updated the plugin topics.
c3130ed to
78a1528
Compare
397e4a9 to
320c30d
Compare
bjhham
left a comment
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.
Added some comments wrt some API changes going in before the release.
Other important changes:
- The
annotatefunction changed todescribeand it now requires@OptIn(ExperimentalKtorApi::class) - I'm intoducing an opposite function
hide()that is called by default for the OpenAPI / Swagger plugins so that you can use the default source and it won't include these endpoints in the resulting document. This can be used for any internal endpoints that you don't want published.
See changes here ktorio/ktor#5300 and here ktorio/ktor#5308
codeSnippets/snippets/openapi-spec-gen/src/main/kotlin/com/example/Application.kt
Outdated
Show resolved
Hide resolved
codeSnippets/snippets/openapi-spec-gen/src/main/kotlin/com/example/Application.kt
Outdated
Show resolved
Hide resolved
codeSnippets/snippets/openapi-spec-gen/src/main/kotlin/com/example/Application.kt
Outdated
Show resolved
Hide resolved
codeSnippets/snippets/openapi-spec-gen/src/main/kotlin/com/example/Application.kt
Outdated
Show resolved
Hide resolved
codeSnippets/snippets/openapi-spec-gen/src/main/kotlin/com/example/Application.kt
Outdated
Show resolved
Hide resolved
|
Thanks for the review @danil-pavlov 🙏 |
KTOR-9165
KTOR-9193