Skip to content

Don’t show ial and aal for now, but leave everything else#7791

Merged
Jeremy1026 merged 1 commit intomainfrom
jcurcio/revert-39d89d
Feb 7, 2023
Merged

Don’t show ial and aal for now, but leave everything else#7791
Jeremy1026 merged 1 commit intomainfrom
jcurcio/revert-39d89d

Conversation

@Jeremy1026
Copy link
Contributor

@Jeremy1026 Jeremy1026 commented Feb 7, 2023

PR #7721 created a problem for a partner (https://gsa-tts.slack.com/archives/C01SU3NB9T3/p1675804144125749) so this will revert the changes made so we can review and decide the best path to implement the changes without causing problems for existing partners.

Copy link
Contributor

@zachmargolis zachmargolis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, do we need to patch this in to production?

… AAL to the OIDC Response Token

Don’t show ial and aal for now, but leave everything else
@Jeremy1026
Copy link
Contributor Author

Yes, but I need to figure out what the valid verbs are for the changelog.

Copy link
Contributor

@orenyk orenyk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit: thank you!

@Jeremy1026 Jeremy1026 force-pushed the jcurcio/revert-39d89d branch from 45eadfe to b50feab Compare February 7, 2023 21:59
@Jeremy1026
Copy link
Contributor Author

Ok, think I got it. Gitlab should complete now. Once it does, I'll merge.

@Jeremy1026 Jeremy1026 merged commit fd14459 into main Feb 7, 2023
@Jeremy1026 Jeremy1026 deleted the jcurcio/revert-39d89d branch February 7, 2023 22:40
zachmargolis pushed a commit that referenced this pull request Feb 7, 2023
… AAL to the OIDC Response Token (#7791)

Don’t show ial and aal for now, but leave everything else

(cherry picked from commit fd14459)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants