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

Turn off consent for pairwise / openid-only scope #350

Closed
martynaslawinska opened this issue Feb 20, 2021 · 0 comments
Closed

Turn off consent for pairwise / openid-only scope #350

martynaslawinska opened this issue Feb 20, 2021 · 0 comments
Assignees
@qbert2k
Labels
comp-jans-auth-server Component affected by issue or PR effort-1 Relative effort required for completion of issue or PR kind-enhancement Issue or PR is an enhancement to an existing functionality priority-4 Minor issue or PR is not relevant to core functions, or relates to the usability of system triaged Issue or PR is fully triaged

Comments

@martynaslawinska
Copy link

martynaslawinska commented Feb 20, 2021 •
edited by nynymike
Loading

nynymike commented on Aug 2, 2019
If a client is configured for pairwise identifiers, and the openid scope is the only scope requested, there is no need to present the consent page, because the AS is not releasing any PII. Also, we should make sure that the claims request is not enabled.
@ghost ghost assigned HemantKMehta Dec 14, 2021
@ghost ghost unassigned HemantKMehta Jan 10, 2022
@ossdhaval ossdhaval transferred this issue from another repository Jan 13, 2022
@ossdhaval ossdhaval added comp-jans-auth-server Component affected by issue or PR effort-1 Relative effort required for completion of issue or PR kind-enhancement Issue or PR is an enhancement to an existing functionality priority-4 Minor issue or PR is not relevant to core functions, or relates to the usability of system ready-for-triage Issue or PR has sufficient details and it is ready for triage review labels Jan 13, 2022
moabu pushed a commit that referenced this issue Jan 13, 2022
@yuriyz
Merge pull request #350 from JanssenProject/issue#342
b568069 
JARM FAPI Test Failed: fapi1-advanced-final-ensure-redirect-uri-in-authorization-request
moabu pushed a commit that referenced this issue Jan 13, 2022
@yuriyz
Merge pull request #350 from JanssenProject/issue#342
88be3c8 
JARM FAPI Test Failed: fapi1-advanced-final-ensure-redirect-uri-in-authorization-request
@qbert2k qbert2k added triaged Issue or PR is fully triaged and removed ready-for-triage Issue or PR has sufficient details and it is ready for triage review labels Jan 24, 2022
qbert2k referenced this issue Jan 25, 2022
@qbert2k
feat(jans-auth-server): turn off consent for pairwise openid-only scope
948520c
@qbert2k qbert2k mentioned this issue Jan 25, 2022
Merged
@qbert2k qbert2k closed this as completed Jan 27, 2022
@yuriyz yuriyz mentioned this issue Jan 31, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@qbert2k qbert2k

Labels
comp-jans-auth-server Component affected by issue or PR effort-1 Relative effort required for completion of issue or PR kind-enhancement Issue or PR is an enhancement to an existing functionality priority-4 Minor issue or PR is not relevant to core functions, or relates to the usability of system triaged Issue or PR is fully triaged
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@ossdhaval @qbert2k @martynaslawinska @HemantKMehta