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.

Sign up for GitHub

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

Jump to bottom

bug(jans-linux-setup): change datatype of jansReqURI, jansClaimRedirectURI and jansAuthorizedOrigins to json of jansClnt table in mysql #1088

Closed
duttarnab opened this issue Mar 21, 2022 · 1 comment
Closed

bug(jans-linux-setup): change datatype of jansReqURI, jansClaimRedirectURI and jansAuthorizedOrigins to json of jansClnt table in mysql #1088

duttarnab opened this issue Mar 21, 2022 · 1 comment
Assignees
@devrimyatar
Labels
comp-jans-linux-setup Component affected by issue or PR

Comments

@duttarnab
Copy link
Contributor

In mysql, multivalued database attributes are mapped to json datatype.

Current Issue
In jansClnt table, claimRedirectUris, authorizedOrigins and requestUris have following datatypes
jansReqURI -->tinytext
jansClaimRedirectURI -->tinytext
jansAuthorizedOrigins -->varchar(64)

Expected change
The jansReqURI, jansClaimRedirectURI and jansAuthorizedOrigins attributes are multivalued. So we need to change the datatype of these columns to json.
@duttarnab duttarnab added the comp-jans-linux-setup Component affected by issue or PR label Mar 21, 2022
devrimyatar added a commit that referenced this issue Mar 21, 2022
@devrimyatar
fix: jans-linux-setup multivalued json mapping (ref: #1088)
b59dd72
@devrimyatar devrimyatar mentioned this issue Mar 21, 2022
Merged
yurem pushed a commit that referenced this issue Mar 23, 2022
@devrimyatar
fix: jans-linux-setup multivalued json mapping (ref: #1088) (#1090)
e3d9dbf 
* fix: jans-linux-setup multivalued json mapping (ref: #1088)

* fix: jans-linux-setup multivalued attributes are json
@devrimyatar
Copy link
Contributor

Implemented

This was referenced May 18, 2022
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@devrimyatar devrimyatar

Labels
comp-jans-linux-setup Component affected by issue or PR
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@devrimyatar @duttarnab