Repository navigation
Feat/public client - #9
Merged
Merged
Conversation
- Update configuration schema and Go structs to include idTokenKey - Support saving ID tokens to .env files when configured - Add --id-token flag to get command for direct output - Add --id-token flag to inspect command for decoding - Make ID token verification non-fatal to increase robustness - Add unit tests for config loading and OIDC token handling - Update README.md with new configuration options and flags
- Refactor `targets` schema to use `type` field instead of `idTokenKey` - Update `inspect` command to intelligently resolve files and keys from the targets list - Improve documentation for multiple target files and fix invalid examples in README - Maintain backward compatibility for simple configuration via `tokenKey` and `idTokenKey`
- Make clientSecret optional and add "none" authMethod - Send client_id in request body without secret for public clients - Reject client_credentials flow when no secret is configured - Add config and OIDC tests for public client behaviour - Document Keycloak public client setup in README
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
✨ Support public OIDC clients (no client secret)
Context
authk required a
clientSecret, so it could not connect to a Keycloak client withClient authentication: Off (public client).
Changes
clientSecretis now optional in the CUE schema (defaults to empty)authMethod: "none"value for public clientsauthMethod: "none"or no secret),client_idis sentin the request body with no
client_secretand noAuthorization: Basicheadererror when no user credentials are configured, without calling the token endpoint
authMethod: "none"is set while a secret is configured, the secret is ignoredand a warning is logged
Configuration example