Skip to content
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

Large value of User Attribute #132

Open
cgeorgilakis opened this issue Nov 1, 2021 · 4 comments
Open

Large value of User Attribute #132

cgeorgilakis opened this issue Nov 1, 2021 · 4 comments
Assignees

Comments

@cgeorgilakis
Copy link

cgeorgilakis commented Nov 1, 2021

We want to save complex attributes from Identity Providers in json format and release them in the same format in Clients ( see ORCID publications). The problem is that these attribute value is over 255 length, which is the database length of value field of user_attribute table.
Related jira issue : https://jira.argo.grnet.gr/browse/RCIAM-656 for publications.

Dev discussion: https://groups.google.com/g/keycloak-dev/c/z6htKyfByd0
Keycloak jira issue has been closed by Stian.

Stian response :

User attributes have been designed to store simple user properties, not complex properties. So an address should be stored as addr.postcode, addr.country, etc, not as {addr.postcode:

{..}
, addr.country: {..}}.

It is too complex for our use case. Or we should find simple value like url to

@cgeorgilakis
Copy link
Author

text field in postgresql

@cgeorgilakis
Copy link
Author

They have added an index in USER_ENTITY table ( correctly). In postgresql index field value maximum size is 8191.
So we have two options either have this size limit either remove index also.

@cgeorgilakis
Copy link
Author

Moreover, tests fail due to index in clob. Other databases do not have text field and liquibase try to create lob field.

@cgeorgilakis
Copy link
Author

cgeorgilakis commented Dec 10, 2021

We increase User Attribute Value to 4000 length and test in https://login-devel.einfra.grnet.gr/auth/ ( release 15.0.2-r1.0.6) .

Some tests in our stage-15.0.2 will be failed due to problem with MySQL with this length. We will ignore them.

@nikosev you could test large User Attributes

cgeorgilakis pushed a commit that referenced this issue May 17, 2024
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

No branches or pull requests

3 participants