8000 Add display to KNOWN_REQ_PARAMS in auth request parser · keycloak keycloak · Discussion #13328 · GitHub

Repository navigation

Add display to KNOWN_REQ_PARAMS in auth request parser #13328

jalavik started this conversation in Ideas

jalavik
Jul 26, 2022

It seems that the display [1] parameter is missing in the list of KNOWN_REQ_PARAMS in https://github.com/keycloak/keycloak/blob/main/services/src/main/java/org/keycloak/protocol/oidc/endpoints/request/AuthzEndpointRequestParser.java#L53-L76. Is there a special reason for this?

display is an expected parameter in the code as it's added to the request object: https://github.com/keycloak/keycloak/blob/main/services/src/main/java/org/keycloak/protocol/oidc/endpoints/request/AuthzEndpointRequestParser.java#L99

But it also ends up in additionalReqParams map and will therefore count towards the limit of custom parameters.

[1] https://openid.net/specs/openid-connect-core-1_0.html#AuthRequest

Replies: 0 comments

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

0