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

403 errors with github auth whitelist #684

Closed
rsignell-usgs opened this issue May 15, 2018 · 1 comment · Fixed by #686
Closed

403 errors with github auth whitelist #684

rsignell-usgs opened this issue May 15, 2018 · 1 comment · Fixed by #686

Comments

@rsignell-usgs
Copy link
Contributor

I'm just capturing this github auth 403 whitelist problem in case other folks deploying find it useful, if I did something incorrect, or if this behavior is unexpected.

On our JH instance at pangeo.esipfed.org, we initially were whitelisting users via the auth: admin section in jupyter-config.yaml, which makes them admin users, and that worked fine -- people on this list had no problem logging in.

  auth:
    admin:
      access: true
      users:
        - jreadey
        - rsignell-usgs

    type: github
    github:
      clientId: "SECRET"
      clientSecret: "SECRET"
      callbackUrl: "http://pangeo.esipfed.org/hub/oauth_callback"
      org_whitelist:
        - "HDFGroup"
        - "pangeo-data"
        - "USGS-CMG"
    scopes:
      - "read:org"

I then decided to add org_whitelist: entries, and after this Helm upgrade, people who logged out to got 403 errors when they tried to log back in.
Looking at the logs for the hub pod:

helm list
kubectl get pods -n esip-dev 
kubectl -n esip-dev logs hub-5647fc9dcd-m86gx

revealed that it said "User rsignell-usgs is not in org whitelist" even though I was listed as an admin, and also was a member of a whitelisted org.

When I went to confirm my membership in the org on github, I saw that my membership in the org was "private", and when I switched it to "public":
2018-05-15_9-31-54
I was able to finally login.

Originally reported on pangeo as pangeo-data/pangeo#256 (comment)

@stevegore
Copy link

Thanks so much for this! Was struggling for a while without this.

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 a pull request may close this issue.

2 participants