Skip to content

OpenID-Connect Plugin - Token Revocation Store Configuration Options#1

Draft
Hockenba wants to merge 2 commits into
masterfrom
feat/openid-connect-stateless-session-revocation
Draft

OpenID-Connect Plugin - Token Revocation Store Configuration Options#1
Hockenba wants to merge 2 commits into
masterfrom
feat/openid-connect-stateless-session-revocation

Conversation

@Hockenba

@Hockenba Hockenba commented Jun 29, 2026

Copy link
Copy Markdown
Collaborator

Adds the schema configurations to the openid-connect plugin needed to configure how lua-resty-session handles the session for token revocation.

@Hockenba Hockenba requested a review from acolomba June 29, 2026 19:42
@Hockenba Hockenba changed the title OpenID-Connect Plugin - Add Token Revocation Store Configuration Options OpenID-Connect Plugin - Token Revocation Store Configuration Options Jun 29, 2026

@acolomba acolomba left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

just one comment about the shape of the configuration

Comment thread apisix/plugins/openid-connect.lua Outdated
return session_conf
end

-- Same redis option shape for session storage (`session.redis`) and the

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

could we not add a "revocation" entry, or even better a "mode" (or something) to session.redis then, vs. having the same config potentially in two places? this would render a bit confusingly in the documentation page, i think

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Addressed.

@Hockenba Hockenba requested a review from acolomba June 30, 2026 14:19
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.

2 participants