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

fix(admin-ui): Storing ACR value is inserting inum instead of displayName #1079

Closed
duttarnab opened this issue Jun 9, 2023 · 0 comments · Fixed by #1080
Closed

fix(admin-ui): Storing ACR value is inserting inum instead of displayName #1079

duttarnab opened this issue Jun 9, 2023 · 0 comments · Fixed by #1080
Assignees
@duttarnab
Labels
comp-admin-ui Component affected by issue or PR kind-bug Issue or PR is a bug in existing functionality

Comments

@duttarnab
Copy link
Contributor

We can select DefaultACR value for each client separately from Admin-UI. Selecting ACR value should insert displayName of that ACR into data source ( i.e. LDAP / mysql ). Instead it's storing DN of that ACR / Person Authentication Script inside Client's data.
Which is resulting error like:

2023-06-09 13:40:27,186 ERROR [qtp2131670196-20] 872064e4-43f8-4fbe-8e69-92e01acf0184 [as.server.authorize.ws.rs.AuthorizeAction] (AuthorizeAction.java:308) - Failed to get CustomScriptConfiguration. auth_step: 1, acr_values: inum=8BAF-80D7,ou=scripts,o=jans

I have recorded a full video what is happening and what is the workaround...
In this video you can see that, we are trying to select Fido2 as default ACR for "modauth.gluu.org" client.
But Admin-UI saving "inum" of that "Fido2" instead of displayName.

Workaround is: manually modify that DefaultACR value.

Here is the recorded video: https://youtu.be/_vLu5f6KY2Q

This issue is similar to another one: GluuFederation/oxTrust#2323
@duttarnab duttarnab added comp-admin-ui Component affected by issue or PR kind-bug Issue or PR is a bug in existing functionality labels Jun 9, 2023
@duttarnab duttarnab self-assigned this Jun 9, 2023
@duttarnab duttarnab mentioned this issue Jun 9, 2023
Closed
duttarnab added a commit that referenced this issue Jun 9, 2023
@duttarnab
fix: Storing ACR value is inserting inum instead of displayName #1079
75228a5
@duttarnab duttarnab mentioned this issue Jun 9, 2023
Merged
jv18creator added a commit that referenced this issue Jun 12, 2023
@jv18creator
Merge pull request #1080 from GluuFederation/admin-ui-issue-1079
6604a99 
fix: Storing ACR value is inserting inum instead of displayName #1079
This was referenced Jun 12, 2023
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@duttarnab duttarnab

Labels
comp-admin-ui Component affected by issue or PR kind-bug Issue or PR is a bug in existing functionality
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: Storing ACR value is inserting inum instead of displayName #1079 GluuFederation/flex
1 participant
@duttarnab