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

Merged
merged 1 commit into from
Jan 20, 2025

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)

Copy link
Contributor

@jshaughn jshaughn left a comment

Choose a reason for hiding this comment

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

Approved but I suggest one small doc change.

crd-docs/crd/kiali.io_kialis.yaml Show resolved Hide resolved
@jshaughn jshaughn merged commit 8a8f6a1 into kiali:master Jan 20, 2025
1 check passed
@jmazzitelli jmazzitelli deleted the require-auth branch January 20, 2025 16:30
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: Done
Development

Successfully merging this pull request may close these issues.

2 participants