Skip to content
This repository was archived by the owner on Jun 1, 2023. It is now read-only.

Needed to complete certification. #42

Merged
merged 9 commits into from
Oct 27, 2021
Merged

Needed to complete certification. #42

merged 9 commits into from
Oct 27, 2021

Conversation

rohe
Copy link
Collaborator

@rohe rohe commented Oct 20, 2021 •
edited
Loading

  • Reread jwks_uri when needed
  • sformat needed to be carried.
  • post_logout_redirect_uri/-s wows
  • refer to the correct type of token.

@rohe rohe requested review from peppelinux and nsklikas October 20, 2021 12:46
peppelinux
peppelinux approved these changes Oct 20, 2021
View reviewed changes
Copy link
Member

@peppelinux peppelinux left a comment

Choose a reason for hiding this comment

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

I read and I approve as it is,
I'll test in oidcop when merged in develop

@rohe rohe merged commit 40fe88a into develop Oct 27, 2021
@rohe rohe deleted the reread_jwks_uri branch October 27, 2021 08:43
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@peppelinux peppelinux peppelinux approved these changes

@nsklikas nsklikas nsklikas approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@rohe @peppelinux @nsklikas