-
Notifications
You must be signed in to change notification settings - Fork 80
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
Enable / Disable Anonymous Access #124
Conversation
@steven-zou can you review this PR? Thanks |
proposal looks good, please check implementation. @steven-zou @reasonerjt |
In regards to the workflow, IMO it's more intuitive if we enhance it at the project level, i.e. when admin sets a project as public, he has additional options to make the project accessible to anonymous users or not. |
@reasonerjt cyber security would ask for the control on the application level, rather than rely on individual's decision. |
So I see the key disagreement is that should the system admin determine the access policy of a project or a project admin. Let's keep this PR pending until we reach agreement. |
Is there any progress on this pr? |
I think the system admin should control whether or not it's possible for a project admin to make things available without authentication, but then leave it up to each project to decide whether they want the project "private", "internal", or "public" (which are the words other similar tools like GitLab use to differentiate these three states). For comparison, GitLab's per-project settings: And the system administrator settings: The reason I think this split makes sense is that the system administrator is typically responsible for determining/enforcing the overall enterprise security policy where the project administrator's responsibilities are going to almost always be a subset of that larger policy. So IMO, we really need both, not just one or the other. |
@steven-zou @cafeliker @tianon did you folks reach agreement on that? looks quite old :) |
looks like merged and and closed as PR! |
cc @qnetter |
This looks like it's been sitting here for a while, but is a pretty important proposal for enterprise. I'd especially support the suggestion @tianon made - for our requirements as system admins we want to be able to restrict anonymous access entirely, as it doesn't fit our use case - but we do want project admins to be able to set their project as 'internally public' to authenticated users, for purposes of sharing images with other teams within our organization etc. But I'd also support this proposal as described, as an MVP, as at the moment the current lack of any option to disable anonymous access is pretty scary. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
support for "private", "internal", or "public"
+1 |
Our company is also interested in the feature. With the docker hub rate limiting we've set up a proxy project and want every user (and project robot accounts in best) to be able to pull images through the proxy. Can I help somehow? |
@tillepille, if interested, I can guide you around the request and PR. |
@Vad1mo thanks! |
I am closing this in favor of #242, which solves the same problem but incorporates the feedback from the community. |
Propose a new feature of allowing administrator enable and disable anonymous access through the system settings page.
More discussion is at goharbor/harbor#10760.
Thanks
ye