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

Set default scopes for github #62

Merged
merged 1 commit into from
Jan 2, 2024
Merged

Set default scopes for github #62

merged 1 commit into from
Jan 2, 2024

Conversation

nsklikas
Copy link
Collaborator

@nsklikas nsklikas commented Dec 15, 2023

IAM-617:

  • When provider_id is github default to the user:email scope.

To test this you need to have a registered Github OAuth application to use for logging in.
To register a Github OAuth application:

  1. Go to https://github.com/settings/applications/new. The application name and homepage URL do not matter, but the Authorization callback URL must be https://<public-traefik-ip>/<model>-kratos/self-service/methods/oidc/callback/github (you can always come back and change this after the bundle is deployed).
  2. Generate a client secret

Now deploy the bundle, refresh with this charm and configure the kratos-external-idp-integrator:

charmcraft pack
juju deploy identity-platform --channel latest/edge --trust
juju refresh kratos-external-idp-integrator --path ./kratos-external-idp-integrator_ubuntu-22.04-amd64.charm
juju config kratos-external-idp-integrator \
    provider=github \
    client_id=<client-id> \
    client_secret=<client-secret> \
    provider_id=github

@nsklikas nsklikas requested a review from a team as a code owner December 15, 2023 13:57
@nsklikas nsklikas merged commit f07f8c3 into main Jan 2, 2024
3 checks passed
@nsklikas nsklikas deleted the IAM-617 branch January 2, 2024 09:20
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