community icon indicating copy to clipboard operation
community copied to clipboard

Enable / disable anonymous access

Open cafeliker opened this issue 4 years ago • 16 comments

Propose a new feature of allowing administrator enable and disable anonymous access through the system settings page.

More discussion is at https://github.com/goharbor/harbor/issues/10760.

Thanks ye

cafeliker avatar Feb 26 '20 06:02 cafeliker

@steven-zou can you review this PR?

Thanks ye

cafeliker avatar Feb 28 '20 03:02 cafeliker

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

xaleeks avatar Feb 29 '20 10:02 xaleeks

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 avatar Apr 13 '20 05:04 reasonerjt

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

cafeliker avatar Apr 15 '20 08:04 cafeliker

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.

reasonerjt avatar Jun 04 '20 02:06 reasonerjt

Is there any progress on this pr?

phin1x avatar Nov 05 '20 14:11 phin1x

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.

tianon avatar Apr 21 '21 14:04 tianon

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

OrlinVasilev avatar Nov 10 '21 12:11 OrlinVasilev

looks like merged and and closed as PR!

OrlinVasilev avatar Nov 10 '21 12:11 OrlinVasilev

cc @qnetter

chlins avatar Jul 11 '22 08:07 chlins

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.

slushysnowman avatar Jan 30 '23 11:01 slushysnowman

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

Vad1mo avatar Jan 30 '23 13:01 Vad1mo

+1

OrlinVasilev avatar Jan 30 '23 19:01 OrlinVasilev

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 avatar Apr 30 '24 20:04 tillepille

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.

Vad1mo avatar May 01 '24 08:05 Vad1mo

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

tillepille avatar May 07 '24 20:05 tillepille

I am closing this in favor of https://github.com/goharbor/community/pull/242, which solves the same problem but incorporates the feedback from the community.

Vad1mo avatar Jun 12 '24 06:06 Vad1mo