Skip to content

Conversation

@julien-nc
Copy link
Member

@julien-nc julien-nc commented Aug 20, 2025

Dispatch a new OCP\Authentication\Events\TokenInvalidatedEvent event in OC\Authentication\Token\PublicKeyTokenProvider::invalidateToken and OC\Authentication\Token\PublicKeyTokenProvider::invalidateTokenById after having actually invalidated a token.

This is one solution to help with nextcloud/user_oidc#1181 in which we need to know when a user session is revoked to terminate the session on the external identity provider side (if needed).

Doc PR: nextcloud/documentation#13522

@julien-nc julien-nc added this to the Nextcloud 32 milestone Aug 20, 2025
@julien-nc julien-nc requested a review from a team as a code owner August 20, 2025 15:57
@julien-nc julien-nc requested review from ArtificialOwl, CarlSchwan and leftybournes and removed request for a team August 20, 2025 15:57
@julien-nc julien-nc added enhancement 3. to review Waiting for reviews feature: authentication pending documentation This pull request needs an associated documentation update labels Aug 20, 2025
@julien-nc julien-nc force-pushed the enh/noid/add-token-invalidated-event branch from fe9a0fd to df3ac5d Compare August 20, 2025 16:06
@julien-nc julien-nc force-pushed the enh/noid/add-token-invalidated-event branch from df3ac5d to 3d36834 Compare August 21, 2025 10:42
@julien-nc julien-nc requested review from susnux August 21, 2025 10:56
@julien-nc julien-nc merged commit 8a8e1c8 into master Aug 21, 2025
198 checks passed
@julien-nc julien-nc deleted the enh/noid/add-token-invalidated-event branch August 21, 2025 12:31
@nextcloud-bot nextcloud-bot mentioned this pull request Aug 22, 2025
@skjnldsv skjnldsv modified the milestones: Nextcloud 32, Nextcloud 33 Sep 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews enhancement feature: authentication pending documentation This pull request needs an associated documentation update

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants