Skip to content
This repository was archived by the owner on Sep 30, 2024. It is now read-only.

Commit 5685bb0

Browse files
authored
allow repo access by default on dotcom (#63367)
Sourcegraph.com only has public repos, so it is safe to allow access to all repositories by all users. This now-deleted code was originally added in https://github.com/sourcegraph/sourcegraph/pull/26345 to support private repositories on dotcom. We do not currently support private repositories on dotcom anymore, and when we do add back that feature, we can figure out how best to do it and will benefit from fewer special-cases for dotcom. ## Test plan CI
1 parent 99968a6 commit 5685bb0

File tree

1 file changed

+0
-7
lines changed

1 file changed

+0
-7
lines changed

internal/authz/register.go

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,6 @@ package authz
33
import (
44
"sync"
55

6-
"github.com/sourcegraph/sourcegraph/internal/dotcom"
76
"github.com/sourcegraph/sourcegraph/internal/testutil"
87
)
98

@@ -35,12 +34,6 @@ func SetProviders(authzAllowByDefault bool, z []Provider) {
3534
authzProviders = z
3635
allowAccessByDefault = authzAllowByDefault
3736

38-
// 🚨 SECURITY: We do not want to allow access by default by any means on
39-
// dotcom.
40-
if dotcom.SourcegraphDotComMode() {
41-
allowAccessByDefault = false
42-
}
43-
4437
authzProvidersReadyOnce.Do(func() {
4538
close(authzProvidersReady)
4639
})

0 commit comments

Comments
 (0)