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

[WIP] OpenID Connect #1230

Closed
wants to merge 52 commits into from
Closed

[WIP] OpenID Connect #1230

wants to merge 52 commits into from

Conversation

hangy
Copy link
Member

@hangy hangy commented Jun 5, 2018

Add an OpenID Connect server, based on OIDC::Lite.

Can be used to add token based authorization to the API, see #1204

Cherry-pick c589869

Conflicts:
	cgi/Users.pm
	cgi/reset_password.pl
Fixed userinfo (missing use statement).
Implemented jwks.pl.
Don't redirect from 'accounts' subdomain (to be used as the OpenID Connect Issuer URL).
@hangy hangy added the security label Jun 5, 2018
@hangy hangy self-assigned this Jun 5, 2018
@TravisBuddy
Copy link

Hey @hangy,
Your changes look good to me!

@hangy
Copy link
Member Author

hangy commented Jun 19, 2018

I'm ambandoning this approach and the oidc branch. It still segfaults on me (even on several different Linux VMs, Docker images, ...), and the OIDC::Lite module is meant to be integrated as a PSGI middleware, and thus cannot be integrated into the current ProductOpener structure easily.

@hangy hangy closed this Jun 19, 2018
@hangy hangy deleted the oidc branch January 25, 2020 19:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants