8000 add KeycloakUserInfo interface by rrodriguezOT · Pull Request #111 · keycloak/keycloak-js · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

add KeycloakUserInfo interface #111

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

rrodriguezOT
Copy link

Closes #16

Closes keycloak#16

Signed-off-by: Ruben Rodriguez <rrodriguez@opentrends.net>
@jonkoops
Copy link
Contributor
jonkoops commented Jun 6, 2025

Nice, this is certainly a type that I have been wanting to expand with actual information. Where did you source the fields that you added here? I would like to verify nothing is missing, or perhaps if there are custom fields that could be added, in which case we should probably add an index type.

@rrodriguezOT
Copy link
Author

I got it from a basic latest keycloak image. I filled some basic fields of admin user and retrieved the user info through keycloak userinfo URL

http://localhost:8080/realms/master/protocol/openid-connect/userinfo
image

Maybe some index type for custom fields are needed you're sure

@jonkoops
Copy link
Contributor

The main issue is that, as far as I know, those fields are not static. This means that only the fields outlined in the spec should be considered. This would mean that only the sub field is guaranteed to be present, and all the rest could vary.

@rrodriguezOT
Copy link
Author

So it will be better to have, at first, an interface just with a fixed property called sub and then another with an index type, isn't it?

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.

keycloak-js: add KeycloakUserInfo interface
2 participants
0