Commit Graph

5 Commits (17d4cc713559c5698ef97139c1e2a57ee241b233)

Author SHA1 Message Date
Jared Scheib f1b9ee5c1b partial revert(43c5afe7) to fix GitHub Enterprise via Generic Oauth
The above commit was over-applied in #3168 to Generic Oauth in
addition to GitHub Oauth based on an assumption. It should only
have been applied to GitHub-specific OAuth. This over-application
introduced a bug where GitHub Enterprise did not work anymore.
2018-05-07 11:27:04 -07:00
Jonathan A. Sternberg 18c3c2320e Fix the github oauth2 implementation with multiple emails
If an account had multiple emails, the current implementation would
always select the first one regardless of any other settings. This fixes
it so it only chooses the primary email address that is verified.

This also fixes the generic oauth2 to require verified and primary to be
true if they are present. If they are not present, they are not
required.
2018-04-11 13:27:15 -05:00
Michael Desa 5da992ae27 Use Generic APIKey for Oauth2 group lookup 2018-02-22 12:54:20 -05:00
Chris Goller 0e72c0d16b Add GENERIC_API_KEY to override of oauth2 responses to support azure 2017-11-21 16:55:03 -06:00
Chris Goller ffd838fc26 Add test for generic oauth2 provider 2017-04-06 15:06:22 -05:00