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

be able to configure /api endpoint to require credentials #865

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jmazzitelli
Copy link
Contributor

When auth.strategy is not anonymous, users can still see the /api endpoint version and config information. This new config allows users to disable that unauthenticated access, requiring users to authenticate with the server in order to see the /api info.

server:
  require_auth: true

server PR: kiali/kiali#8055

When auth.strategy is not anonymous, users can still see the /api endpoint version and config information.
This new config allows users to disable that unauthenticated access, requiring users to authenticate with the server in order to see the /api info.
@jmazzitelli jmazzitelli added the requires server PR A PR requires additional changes in the backend code. label Jan 13, 2025
@jmazzitelli jmazzitelli self-assigned this Jan 13, 2025
@jmazzitelli
Copy link
Contributor Author

for test procedures, see kiali/kiali#8055 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
requires server PR A PR requires additional changes in the backend code.
Projects
Status: 👀 In review
Development

Successfully merging this pull request may close these issues.

1 participant