Skip to content

[#1015] docs: Add OAS of security schemes#1048

Merged
jerryshao merged 1 commit intoapache:mainfrom
mchades:issue-1015
Dec 9, 2023
Merged

[#1015] docs: Add OAS of security schemes#1048
jerryshao merged 1 commit intoapache:mainfrom
mchades:issue-1015

Conversation

@mchades
Copy link
Contributor

@mchades mchades commented Dec 9, 2023

What changes were proposed in this pull request?

Add OAS of security schemes

Why are the changes needed?

Fix: #1015

Does this PR introduce any user-facing change?

no

How was this patch tested?

project build successfully

@mchades mchades self-assigned this Dec 9, 2023
BearerAuth:

OAuth2WithJWT:
type: oauth2
Copy link
Contributor

Choose a reason for hiding this comment

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

Does this have schema?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

no, scheme applies to http type, and flows applies to oauth2 type, see OAS

Copy link
Contributor

Choose a reason for hiding this comment

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

ok.

@github-actions
Copy link

github-actions bot commented Dec 9, 2023

Code Coverage Report

Overall Project 65.94% 🟢

There is no coverage information present for the Files changed

Copy link
Contributor

@qqqttt123 qqqttt123 left a comment

Choose a reason for hiding this comment

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

LGTM.

@jerryshao jerryshao merged commit 47bbf7f into apache:main Dec 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Subtask] Add OAS of security schemes

3 participants