-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Conversation
Oh, saw the comment on the other bug. Need to use 'authenticated_emails' file. Got it working after adding one in with my email. But since github authentication can already restrict to an organization, why is that necessary? |
@mightyguava thanks for commenting here, So i think this is just the conflation of a few documentation things that are confusing (and the fact that github is the only provide that supports role based authentication means that's a little un-documented so far). Using github org validation doesn't by default allow all email addresses it gets to be considered valid. In addition there isn't a way to wildcard allow all email addresses yet (#91). In addition the option to whitelist a whole domain is currently misleadingly named If you feel there is something here beyond those please re-open and let me know |
Thanks for the quick reply. So if I'm understanding correctly, If I set When I turned on
My email is no longer showing up in the logs here, so I'm assuming that it isn't parsing or getting my email properly. |
Okay, I think the issue is here https://github.com/bitly/oauth2_proxy/blob/master/providers/github.go#L83 If I specify Org but not Team, the github provider will still try to check against teams. However, if I'm not part of any teams at all, the loop at line 83 never runs, and I fail the check. I think there should be a check before the loop to return |
Ah, reading the code more, the problem is more that there's no support to only check a user's organization? |
Thanks for the follow ups. I'll try to reproduce this soon |
@mightyguava I've pushed up a change that should handle the case you have where a user is part of an org but not a team. Can you validate that this works for you? |
Github provider "Invalid Account"
Yup. It is working for me. Thanks! |
Yay! Thanks for the confirmation. |
Still seeing this issue. To clarify, the desired behavior that I'd like to see, and that I think OP was hoping for, was that, if I specify a github org, that means I'm saying I want all users who have access to that org to have access here, without specifying an email file. I believe this is how the google apps integration works if you specify a domain, right? |
However, I'm using the latest stable release, not sure if that has the 13e8292 |
I'm having trouble trying out oauth2 proxy with the Github provider as well. I'm able to get to the Github login page, log in, and then get redirected to the callback, but then I end up at this page:
The URL looks like:
https://myapp.com/oauth2/callback?code=<some_code>&state=%2Foauth2
Here's my configuration
And logs
Don't have the unmarshalling problem from #108.