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

Adjust order and behavior of client auth settings #1543

Closed
willow9886 opened this issue Mar 5, 2019 · 1 comment
Closed

Adjust order and behavior of client auth settings #1543

willow9886 opened this issue Mar 5, 2019 · 1 comment
Assignees
@syntrydy
Labels
exclude from release notes issues found during QA that should be excluded from release notes Needs Documentation Issue that requires documentation to be added in Gluu docs Needs screenshot Resolved issue requires a new or updated screenshot for the documentation
Milestone
3.1.6

Comments

@willow9886
Copy link
Contributor

willow9886 commented Mar 5, 2019 •
edited
Loading

  1. We should swap the order of the two authorization fields on the client page. Pre-Authorization should be first, and Persist Client Authorization should follow.

  2. In addition, the current behavior is incorrect. Currently when Pre-Auth == True, Persis Auth is set to False, and it cannot be adjusted.

The correct behavior should be the following:

  • When Pre-Auth == True, Persist Auth should also == true, and the field should be disabled.

  • When Pre-Auth == False, Persist Auth should be configurable True or False.

image
@willow9886 willow9886 added the exclude from release notes issues found during QA that should be excluded from release notes label Mar 5, 2019
@willow9886 willow9886 added this to the 3.1.6 milestone Mar 5, 2019
syntrydy added a commit that referenced this issue Mar 5, 2019
@syntrydy
Adjust order and behavior of client auth settings #1543
c19bb18
syntrydy added a commit that referenced this issue Mar 5, 2019
@syntrydy
Merge pull request #1548 from /issues/1543
498233a 
Adjust order and behavior of client auth settings #1543
@syntrydy syntrydy closed this as completed Mar 5, 2019
@devrimyatar
Copy link

I
screenshot_2019-03-06_22-41-08
checked the issue if fixed

@willow9886 willow9886 added Needs Documentation Issue that requires documentation to be added in Gluu docs Needs screenshot Resolved issue requires a new or updated screenshot for the documentation labels Mar 7, 2019
shmorri added a commit to GluuFederation/docs-ce-prod that referenced this issue Mar 7, 2019
@shmorri
Updated Pre-Auth and Persist Auth settings
511c3cc 
According to oxTrust issue [#1543](GluuFederation/oxTrust#1543)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@syntrydy syntrydy

Labels
exclude from release notes issues found during QA that should be excluded from release notes Needs Documentation Issue that requires documentation to be added in Gluu docs Needs screenshot Resolved issue requires a new or updated screenshot for the documentation
Projects
None yet
Milestone
3.1.6
Development

No branches or pull requests
3 participants
@willow9886 @devrimyatar @syntrydy