Skip to content

oauth2 login disabled if registration is disabled #2768

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Closed
1 of 3 tasks
yannick opened this issue Oct 23, 2017 · 2 comments · Fixed by #3910
Closed
1 of 3 tasks

oauth2 login disabled if registration is disabled #2768

yannick opened this issue Oct 23, 2017 · 2 comments · Fixed by #3910
Labels
type/feature Completely new functionality. Can only be merged if feature freeze is not active.
Milestone

Comments

@yannick
Copy link

yannick commented Oct 23, 2017

  • Gitea version (or commit ref):

  • Git version: 1.1.0+655-ga257f88a built with: bindata, sqlite

  • Operating system: macOS

  • Database (use [x]):

    • [ x] SQLite
  • Can you reproduce the bug at https://try.gitea.io:

    • Yes (provide example URL)
    • No
    • Not relevant

Description

when i disable registration in config i am not able to login via oauth providers unless the user registerd before.

it should be possible to disable manual registration and only allow oauth2 providers to login/register
also it should be possible to limit e.g. google oauth logins to specific domains only

@lafriks lafriks added the type/feature Completely new functionality. Can only be merged if feature freeze is not active. label Oct 23, 2017
@lafriks lafriks added this to the 1.x.x milestone Oct 23, 2017
@gjsman
Copy link

gjsman commented Nov 6, 2017

Yes, I really want this. I would love if I could use an OAuth Provider (Auth0.com in this case), and turn off the regular login form, allowing only people who # successfully through OAuth to register. 👍

@gjsman
Copy link

gjsman commented Nov 6, 2017

In the settings, there are settings for:

  • Disabling Registration
  • Enabling OpenID Login
  • Enabling OpenID Registration

Obviously, the OAuth thing (when added as an "Authentication" doesn't use the OpenID values). In my opinion, we should have an additional setting, in the Authentication settings, which overrides the "Disable Registration" option specifically for that authentication.

# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
type/feature Completely new functionality. Can only be merged if feature freeze is not active.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants