Skip to content

KEYCLOAK-10761 Add support for Access-Token encryption#7340

Closed
thomasdarimont wants to merge 3 commits intokeycloak:masterfrom
thomasdarimont:issue/KEYCLOAK-XXX-Add-Support-for-AccessToken-Encryption
Closed

KEYCLOAK-10761 Add support for Access-Token encryption#7340
thomasdarimont wants to merge 3 commits intokeycloak:masterfrom
thomasdarimont:issue/KEYCLOAK-XXX-Add-Support-for-AccessToken-Encryption

Conversation

@thomasdarimont
Copy link
Copy Markdown
Contributor

@thomasdarimont thomasdarimont commented Aug 15, 2020

  • Leveraged existing IDToken encryption support for AccessToken encryption.
    Note: Signed & Encrypted JWTs need cty: "JWT" (the content-type) in the JWE header.

  • Added appropriate client configuration options to "Advanced OIDC Client Settings" in admin-console.
    We now can configure signature / encryption algorithms and a JWE keystore to support decryption of JWEs for UserInfoEndpoint and TokenIntrospection endpoint.
    See: draft-ietf-oauth-jwt-introspection-response-09
    Under "3. Resource Server Management" Page 2.:

...
   To support encrypted token introspection response JWTs, the
   authorization server MUST also be provided with the respective
   resource server encryption keys and algorithms.
...
  • Enhanced TokenVerifier to support JWE decryption.

A comprehensive example can be found here: https://github.com/thomasdarimont/spring-boot-keycloak-jwe-example
Note: If you want encryption support for UserInfo and Token Introspection endpoint, you need to configure the
JWE Keystore and respective Passwords in the Advanced OIDC Settings for the jweclient in the example.

This is a new attempt in the vein of the outdated PR #6796

Leveraged existing IDToken encryption support for AccessToken encryption.
Note: Signed & Encrypted JWTs need cty: "JWT" (the content-type) in the JWE header.
@thomasdarimont thomasdarimont force-pushed the issue/KEYCLOAK-XXX-Add-Support-for-AccessToken-Encryption branch 2 times, most recently from afa7638 to 0cc60d2 Compare August 15, 2020 11:12
@thomasdarimont
Copy link
Copy Markdown
Contributor Author

Superseded by #7341

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant