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

Allow users of a specific GitHub organization to login #642

Closed
wants to merge 5 commits into from

Conversation

pyro2927
Copy link
Contributor

If GITHUB_ORG_ID is set in ENV, and someone attempts to login via GitHub without a prior account, one is automatically created for them. This is both convenient and secure, by not necessitating an administrator to create accounts for every user before they can login. It also restricts via the organization ID to prevent non-organization members from creating accounts. After creating their account they can choose to edit their profile to an email different than their GitHub one.

@pyro2927
Copy link
Contributor Author

pyro2927 commented Feb 2, 2014

Updating github_api gem, as it's been updated to support faraday 9+ with piotrmurach/github#148.

@arthurnn
Copy link
Member

arthurnn commented Feb 2, 2014

Why do we need the github gem? could we just use the octokit one as already included.

@pyro2927
Copy link
Contributor Author

pyro2927 commented Feb 2, 2014

Yea, I somehow missed that Octokit was in there. I'll replace the current github_api code with octokit.

@pyro2927
Copy link
Contributor Author

pyro2927 commented Feb 2, 2014

@arthurnn, would you prefer to have the last commit in it's own PR for cleanliness?

@arthurnn
Copy link
Member

arthurnn commented Feb 2, 2014

If you could squash all commits in one, and make ci pass would be awesome.. thanks

@pyro2927
Copy link
Contributor Author

pyro2927 commented Feb 2, 2014

Closing this in favor of #644.

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.

None yet

2 participants