Skip to content

Add better auth support #74

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 6 commits into from
Dec 6, 2021
Merged

Add better auth support #74

merged 6 commits into from
Dec 6, 2021

Conversation

bourdakos1
Copy link
Member

@bourdakos1 bourdakos1 commented Dec 5, 2021 •
edited
Loading

This PR brings a few enhancements to authentication:

  • The demo panel now allows you to choose the security scheme from a dropdown that is populated by the OpenAPI definition (only showing the dropdown if more than one is listed)
  • Adds support for using multiple auth modes simultaneously (Eg: (BearerAuth) OR (ApiKeyAuth AND BasicAuth))
  • Adds an authPersistence option to themeConfig.api. Defaults to "localStorage", can be set to false to disable or sessionStorage to only persist while the window is open.

Fixes: #67

@bourdakos1 bourdakos1 marked this pull request as ready for review December 6, 2021 02:53
@bourdakos1
Copy link
Member Author

FYI @afterthought I think this adds one of the features you were interested in working on:

An option to persist credentials (local storage) to make interactive API exploration a little more user friendly

@bourdakos1 bourdakos1 merged commit 6edb4a8 into cloud-annotations:main Dec 6, 2021
@bourdakos1 bourdakos1 deleted the auth-toggle branch December 6, 2021 04:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Authorization: config or default to Bearer token approach
1 participant
@bourdakos1