Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added support for anonymous usage to chain authentication plugins #103

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

JakeCodeStuff
Copy link

Based on Kong oauth2 official plugin

It was tested/developed against Kong 14.1 CE (as that is what I am using). You/I can easily update to the latest 1.X. You guys should probably version your plugin based in the Kong version supported.

To update to support latest kong 1.x, reference Kong official oath2 plugin for syntax (its mostly differences in kong built-in libraries.

@csmcgee
Copy link

csmcgee commented Jul 25, 2019

EDIT: definitely left my original comment on the wrong PR....

I'm assuming this won't be merged until it passes testing?

I'm currently experimenting with the plugin and believe this functionality of being able to logically 'or' (chain multiple oidc plugins) will be supported with this change.

Per kong documentation Multiple Auth

@Trojan295
Copy link
Contributor

I'm no longer maintaining the repository. Don't know, if there's anyone else..

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.

3 participants