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

[16.0] base_rest: Define the oauth2 redirect url into swagger ui #379

Merged
merged 1 commit into from
Oct 7, 2023

Conversation

lmignon
Copy link
Contributor

@lmignon lmignon commented Oct 7, 2023

No description provided.

@lmignon lmignon force-pushed the 16.0-base-rest-swagger-oauth2 branch from 95d55a2 to 2e878c5 Compare October 7, 2023 06:23
…er ui

Add support for oauth2 security scheme in the Swagger UI. If your openapi
specification contains a security scheme of type oauth2, the Swagger UI will
display a login button in the top right corner. In order to finalize the
login process, a redirect URL must be provided when initializing the Swagger
UI. The Swagger UI is now initialized with a `oauth2RedirectUrl` option that
references a oauth2-redirect.html file provided by the swagger-ui lib and served
by the current addon.
@lmignon lmignon force-pushed the 16.0-base-rest-swagger-oauth2 branch from 2e878c5 to df102d1 Compare October 7, 2023 06:47
@lmignon
Copy link
Contributor Author

lmignon commented Oct 7, 2023

ping @sbidoul

@sbidoul
Copy link
Member

sbidoul commented Oct 7, 2023

👍

/ocabot merge patch

@OCA-git-bot
Copy link
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 16.0-ocabot-merge-pr-379-by-sbidoul-bump-patch, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit fd6927d into OCA:16.0 Oct 7, 2023
5 of 6 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at dd3f433. Thanks a lot for contributing to OCA. ❤️

@sbidoul sbidoul deleted the 16.0-base-rest-swagger-oauth2 branch October 7, 2023 09:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants