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

Backfill oidc scopes #281

Merged
merged 1 commit into from
Jun 29, 2022
Merged

Backfill oidc scopes #281

merged 1 commit into from
Jun 29, 2022

Conversation

michaeljguarino
Copy link
Member

Summary

We'll need this to properly support refresh tokens via hydra

Test Plan

added unit test

Checklist

  • If required, I have updated the Plural documentation accordingly.
  • I have added tests to cover my changes.
  • I have added a meaningful title and summary to convey the impact of this PR to a user.
  • I have added relevant labels to this PR to help with categorization for release notes.

Fixes #279

@michaeljguarino
Backfill oidc scopes
649266e 
We'll need this to properly support refresh tokens via hydra
@michaeljguarino michaeljguarino added the bug Something isn't working label Jun 29, 2022
@michaeljguarino michaeljguarino merged commit 27ee434 into master Jun 29, 2022
@michaeljguarino michaeljguarino deleted the backfill-hydra-clients branch June 29, 2022 16:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
bug Something isn't working
Projects
Plural API Workstream
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

OIDC consent uses harcoded scopes and doesn't allow remembering consent
1 participant
@michaeljguarino