Skip to content
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

[2201.9.x] Generate and host SwaggerUI for the generated OpenAPI specification as a built-in resource #2043

Merged
merged 4 commits into from
Jun 14, 2024

Conversation

ayeshLK
Copy link
Member

@ayeshLK ayeshLK commented Jun 14, 2024

Purpose

$subject

Fixes: #6622

Examples

configurable int port = 9090;

# A fake mountain resort
@openapi:ServiceInfo { embed: true }
@http:ServiceConfig { mediaTypeSubtypePrefix: "vnd.snowpeak.resort", cors: { allowOrigins: ["*"] }}
service /snowpeak on new http:Listener(port) {
  // ...
}

swagger-ui-sample-ext

  • Following is a sample screenshot for a service which contains an invalid OpenAPI definition.

swagger-ui-error

Checklist

  • Linked to an issue
  • Updated the changelog
  • Added tests
  • Updated the spec
  • Checked native-image compatibility
  • Checked the impact on OpenAPI generation

Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

@TharmiganK TharmiganK added the Skip GraalVM Check This will skip the GraalVM compatibility check label Jun 14, 2024
Copy link
Contributor

@TharmiganK TharmiganK left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@dulajdilshan dulajdilshan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is the reason for this random string value dygixywsw?

@TharmiganK
Copy link
Contributor

What is the reason for this random string value dygixywsw?

This is to make the URL unique. Please refer to this comment: ballerina-platform/ballerina-library#1616 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Skip GraalVM Check This will skip the GraalVM compatibility check
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants