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

Enable / Disable Anonymous Access #124

Closed
wants to merge 3 commits into from
Closed

Conversation

cafeliker
Copy link

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

@cafeliker
Copy link
Author

@steven-zou can you review this PR?

Thanks
ye

@xaleeks
Copy link
Contributor

xaleeks commented Feb 29, 2020

proposal looks good, please check implementation. @steven-zou @reasonerjt

@reasonerjt
Copy link
Contributor

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.

@cafeliker
Copy link
Author

@reasonerjt cyber security would ask for the control on the application level, rather than rely on individual's decision.

@reasonerjt
Copy link
Contributor

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.

@phin1x
Copy link

phin1x commented Nov 5, 2020

Is there any progress on this pr?

@tianon
Copy link
Member

tianon commented Apr 21, 2021

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:

image

And the system administrator settings:

image

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.

@OrlinVasilev
Copy link
Member

@steven-zou @cafeliker @tianon did you folks reach agreement on that? looks quite old :)

@OrlinVasilev
Copy link
Member

looks like merged and and closed as PR!

@chlins
Copy link
Member

chlins commented Jul 11, 2022

cc @qnetter

@OrlinVasilev OrlinVasilev assigned wy65701436 and qnetter and unassigned reasonerjt Dec 12, 2022
@slushysnowman
Copy link

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.

@Vad1mo
Copy link
Member

Vad1mo commented Jan 30, 2023

I also support @tianon proposal much like GitLab dues it. Private, internal, public.

Copy link
Member

@Vad1mo Vad1mo left a 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"

@OrlinVasilev
Copy link
Member

+1

@OrlinVasilev OrlinVasilev requested review from a team as code owners January 30, 2023 19:38
@tillepille
Copy link

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?

@Vad1mo
Copy link
Member

Vad1mo commented May 1, 2024

Can I help somehow?
we need someone who takes the lead in this proposal and its implementation.

@tillepille, if interested, I can guide you around the request and PR.

@tillepille
Copy link

tillepille commented May 7, 2024

@Vad1mo thanks!
I can do my best to lead the proposal but for implementing... I am not really proficient in Go, I fear 😬

@Vad1mo Vad1mo changed the title Enable / disable anonymous access Control Access to Harbor with Visibility Levels Jun 11, 2024
@Vad1mo Vad1mo changed the title Control Access to Harbor with Visibility Levels Enable / Disable Anonymous Access Jun 11, 2024
@Vad1mo
Copy link
Member

Vad1mo commented Jun 12, 2024

I am closing this in favor of #242, which solves the same problem but incorporates the feedback from the community.

@Vad1mo Vad1mo closed this Jun 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.