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

OIDC integration #130

Merged
merged 13 commits into from
Sep 27, 2022
Merged

Conversation

jverswijver
Copy link
Contributor

@jverswijver jverswijver commented Sep 26, 2022

fix #125

Copy link
Collaborator

@guzman-raphael guzman-raphael left a comment

Choose a reason for hiding this comment

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

@jverswijver Looks like you are getting the hang of this OIDC business! 👮

Suggesting a way to make the codebase a bit more clear.

docker-compose-dev.yaml Outdated Show resolved Hide resolved
pharus/server.py Outdated Show resolved Hide resolved
pharus/server.py Outdated Show resolved Hide resolved
pharus/interface.py Outdated Show resolved Hide resolved
@jverswijver
Copy link
Contributor Author

fix #125

@guzman-raphael guzman-raphael merged commit 63ff9ad into datajoint:master Sep 27, 2022
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.

Upgrade Authorizaiton Scheme
2 participants