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

Support federated login for guest users (OpenID) #185

Closed
stevenroose opened this issue Nov 16, 2016 · 19 comments · Fixed by #618
Closed

Support federated login for guest users (OpenID) #185

stevenroose opened this issue Nov 16, 2016 · 19 comments · Fixed by #618
Labels
type/proposal The new feature has not been accepted yet but needs to be discussed first.
Milestone

Comments

@stevenroose
Copy link

stevenroose commented Nov 16, 2016

From @stevenroose on May 26, 2015 9:5

It bothers many that they have to enter a full form of personal information in order to just file a bug or make a feature request.
OpenID support can fix this and allows guest users to login with they GitHub of Google account f.e.

Ideally, it would also be possible to restrict these "guest"/external users to only be able to make create issues and pull requests and not have their own repositories.

Copied from original issue: gogs/gogs#1297

--- Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/39298589-support-federated-login-for-guest-users-openid?utm_campaign=plugin&utm_content=tracker%2F47456670&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F47456670&utm_medium=issues&utm_source=github).
@sbrl
Copy link

sbrl commented Nov 16, 2016

Waaaaat? I don't understand what's going on here. What's gitea? Why has it been forked? Is gogs still a thing?

To answer [this comment], I don't think I can since I don't have go installed, I don't know go, I don't understand the go package system, and I don't understand openid 😕 😞

???

@tboerger tboerger added the type/proposal The new feature has not been accepted yet but needs to be discussed first. label Nov 16, 2016
@tboerger tboerger added this to the 1.x.x milestone Nov 16, 2016
@tboerger
Copy link
Member

@stevenroose you shouldn't copy everything, a single reference to the issue should be fine.

@strk
Copy link
Member

strk commented Nov 17, 2016

@sbrl gitea is a fork of gogs made to avoid development being blocked by a single person in charge of accepting contributions. Management of gitea is in the hands of 3 owners elected yearly by a growing number of maintainers (currently 10). See the CONTRIBUTING.md file for further details.

Binary packages, for those who don't want to build from sources, are in the works, see #31

@sbrl
Copy link

sbrl commented Nov 17, 2016

Ah right! I might switch when a full release is made then.

@bkcsoft
Copy link
Member

bkcsoft commented Nov 23, 2016

@tboerger One can always clean it up 😉 (Like I just did)

@tscs37
Copy link

tscs37 commented Dec 10, 2016

I'm in favor of adding basic OpenID support and like to also propose OpenID Connect support.

That would allow arbitrary federation from any provider that supports OpenID Connect.

@strk
Copy link
Member

strk commented Dec 10, 2016

@tscs37 are you up to try at moving some steps toward the implementation of one or the other ?
I've moved the first step for OpenID but it's far from ready (mostly needs UX design), see #271

As per OpenID Connect, do you know of any consumer implementation that does support arbitrary provider support ? I'm asking because I don't and would like to spread the word about its existance.

@stevenroose
Copy link
Author

@strk StackExchange has it (prob not open source). I thought the OSQA had it too, that's an old open-source clone of SE. Or is that basic OpenID?

@strk
Copy link
Member

strk commented Dec 12, 2016 via email

@stevenroose
Copy link
Author

Just to copy from @strk 's PR:

The big advantage I would see from having OpenID is as a way of users to login without having been registered. Just like systems like Discuss allow anyone with an OpenID (even though Discuss requires this to be from a small set of providers) to make contributions in the form of issues or (federated) pull requests.

A possible setup that would benefit greatly from OpenID is the case of single-user usage. Gitea as a self-hosted Git homepage in which only one user has commit access, while he can allow anyone to open issues or request pulls with their OpenID (and a CAPTCHA ofc).

@strk
Copy link
Member

strk commented Dec 14, 2016 via email

@lunny lunny modified the milestones: 1.2.0, 1.x.x Mar 17, 2017
@stevenroose
Copy link
Author

Does #816 allow guest users in some way?

@strk
Copy link
Member

strk commented Mar 20, 2017

@stevenroose I guess you meant #618 (OpenID). It depends on what you mean by "guest users".
Logging in via OpenID still needs you to register as a local user, and confirm your email if the service is configured like that.

The registration step is needed to have a unique identifier on the specific node (rather than using the full OpenID URI as an identifier). Also the code to deal with permissions and teams invitations and such are all based on a local "UserID", so registration is also needed for that (to assign an UserID to an OpenID account).

Doing the registration step also lets you associate multiple OpenID URIs to a single account.

BTW, @stevenroose as you mentioned stackexchange, it also supports OpenID-2.0 so you do can login to Gitea now with "openid.stackexchange.com" as your OpenID URI :)

@sbrl
Copy link

sbrl commented Mar 20, 2017

@strk Awesome! So can I restrict auto-registered account to not be able to create repositories too?

@strk
Copy link
Member

strk commented Mar 22, 2017

To my knowledge there's currently no support to specify default permissions for new users, but it'll be great to add that feature !

@stevenroose
Copy link
Author

@strk @sbrl, yeah that's what I meant with "guest users". Those that only can make issues and submit federated PRs. I.e. not being able to take resources, so that you don't have to trust them not to DOS down your Gitea..

@strk
Copy link
Member

strk commented Mar 22, 2017 via email

@sbrl
Copy link

sbrl commented Mar 23, 2017

I'd allow the admin to set the amount of space allowed on a per-group basis or something. My main concern is losing all my server's (rather limited) hard drive space.

And can I migrate to gitea from gogs version 0.10.1.0228? I missed the first release of gitea, and according to the website it's not an explicitly supported upgrade path.

@strk
Copy link
Member

strk commented Mar 23, 2017

@sbrl see also #1377 for setting users limits (and use the forum or IRC chat or file another issue to know about upgrade)

@go-gitea go-gitea locked and limited conversation to collaborators Nov 23, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
type/proposal The new feature has not been accepted yet but needs to be discussed first.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants