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

docs: add prompt to refresh token requirements#148

Merged
rohe merged 1 commit intoIdentityPython:masterfrom
melanger:patch-1
Nov 6, 2021
Merged

docs: add prompt to refresh token requirements#148
rohe merged 1 commit intoIdentityPython:masterfrom
melanger:patch-1

Conversation

@melanger
Copy link
Contributor

@melanger melanger commented Nov 1, 2021

@peppelinux
Copy link
Member

Good contribution, thank you!
@rohe we know that's something that belong to oidc certification requirements. @melanger may we have to say this in the docs?

I mean, that's not am oidcop requirement but a oidc specification one

@rohe
Copy link
Collaborator

rohe commented Nov 1, 2021

True, it's an OIDC specification requirement. Though there are caveats, as always. :-/

@peppelinux peppelinux requested a review from rohe November 1, 2021 18:37
@peppelinux
Copy link
Member

Ok I think it's worth of it to specify this behaviour also in the documentation, specially for legacy clients and consumers that would think that oidcop has some strange behaviour :)

I would like just to underline that's something required from oidc certification, and not a free initiative of the oidcop developers :)

@melanger
Copy link
Contributor Author

melanger commented Nov 2, 2021

Ok I think it's worth of it to specify this behaviour also in the documentation, specially for legacy clients and consumers that would think that oidcop has some strange behaviour :)

I would like just to underline that's something required from oidc certification, and not a free initiative of the oidcop developers :)

Note added.

@peppelinux peppelinux self-requested a review November 2, 2021 15:15
@peppelinux
Copy link
Member

@rohe this is an easy shot, once merged the official docs will be updated automatically (because it's on master branch)

@rohe rohe merged commit bd975e8 into IdentityPython:master Nov 6, 2021
@melanger melanger deleted the patch-1 branch November 6, 2021 18:38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants