Skip to content

KEYCLOAK-19495 Realm key(s) uses - Allowing multivalue fields#8539

Closed
laskasn wants to merge 1 commit intokeycloak:mainfrom
eosc-kc:110-keyuses
Closed

KEYCLOAK-19495 Realm key(s) uses - Allowing multivalue fields#8539
laskasn wants to merge 1 commit intokeycloak:mainfrom
eosc-kc:110-keyuses

Conversation

@laskasn
Copy link
Copy Markdown
Contributor

@laskasn laskasn commented Oct 6, 2021

This is the first pull request as described here

@stianst stianst changed the title KEYCLOAK-19495 KEYCLOAK-19495 Realm key(s) uses - Allowing multivalue fields Oct 7, 2021
@mposolda
Copy link
Copy Markdown
Contributor

mposolda commented Aug 7, 2023

@laskasn CC @cgeorgilakis There are some changes in latest Keycloak related to keys and key use etc. Just to confirm, could you please doublecheck if latest Keycloak 22.0.1 doesn't already address your use-case?

@cgeorgilakis
Copy link
Copy Markdown
Contributor

@laskasn CC @cgeorgilakis There are some changes in latest Keycloak related to keys and key use etc. Just to confirm, could you please doublecheck if latest Keycloak 22.0.1 doesn't already address your use-case?

@mposolda as I see, what we want is not supported yet in Keycloak.
Our summarized proposal for this PR is :

So, for the first one, we change the exclusiveness of each key’s use for either “enc” or “sig”, and -instead- assign a list of uses for each key i.e. [“enc”, “sig”].

If you want to proceed with this PR, we need to rebase it and move ui changes from old admin console to new admin console.

@stianst
Copy link
Copy Markdown
Contributor

stianst commented Mar 15, 2024

I'm going to close this as I think it adds unnecessary complexity. If anyone really wants to use the same keys for enc and sig that can be done by registering the keys twice, or through a custom key provider.

@stianst stianst closed this Mar 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants