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.

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

Put Swagger behind auth #439

Open
UncleGedd opened this issue Oct 11, 2024 · 3 comments
Open

Put Swagger behind auth #439

UncleGedd opened this issue Oct 11, 2024 · 3 comments
Labels
tech debt Record something that should be investigated or refactored in the future.

Comments

@UncleGedd
Copy link
Contributor

Describe what should be investigated or refactored

Ensure that we have some auth in front of Swagger when running locally
@UncleGedd UncleGedd added the tech debt Record something that should be investigated or refactored in the future. label Oct 11, 2024
@TristanHoladay
Copy link
Contributor

@UncleGedd this was handled in your auth refactor PR right?

@TristanHoladay TristanHoladay self-assigned this Oct 17, 2024
@UncleGedd
Copy link
Contributor Author

Nah there was a bug, fixed here: #466

@UncleGedd
Copy link
Contributor Author

^ noting that #466 handles local auth, but we also need to handle in-cluster auth

@TristanHoladay TristanHoladay removed their assignment Nov 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
tech debt Record something that should be investigated or refactored in the future.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@TristanHoladay @UncleGedd