diff options
| author | Madeline Kalil <mkalil@google.com> | 2026-04-08 10:45:56 -0400 |
|---|---|---|
| committer | Madeline Kalil <mkalil@google.com> | 2026-04-08 09:14:28 -0700 |
| commit | e09b3908a3542531bb1a84b28406102957861650 (patch) | |
| tree | 32c06873ccc2c2e7271782c63bff859413c18324 | |
| parent | 5c3cacb5982231e5d4acd75f6c4cf1bb8f7ba411 (diff) | |
| download | go-x-pkgsite-e09b3908a3542531bb1a84b28406102957861650.tar.xz | |
internal/frontend: remove local map of case sensitive excluded paths
go.dev/cl/761521 introduced an exact: prefix used to specify case sensitive paths that need to be excluded.
We can now remove the map that previously handled case sensitive exclusion. A separate CL merges the outstanding case sensitive excluded paths into excluded.txt.
For golang/go#77822
Change-Id: Ic4d15ae3deda9323981fb5ceaa5667522278960a
Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/764080
kokoro-CI: kokoro <noreply+kokoro@google.com>
Reviewed-by: Ethan Lee <ethanalee@google.com>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
| -rw-r--r-- | internal/frontend/details.go | 14 |
1 files changed, 0 insertions, 14 deletions
diff --git a/internal/frontend/details.go b/internal/frontend/details.go index f1438de4..0323f040 100644 --- a/internal/frontend/details.go +++ b/internal/frontend/details.go @@ -87,9 +87,6 @@ func stdlibRedirectURL(fullPath string) string { } func checkExcluded(ctx context.Context, ds internal.DataSource, fullPath, version string) error { - if caseSensitiveExcludedPaths[fullPath] { - return &serrors.ServerError{Status: http.StatusNotFound} - } db, ok := ds.(internal.PostgresDB) if !ok { return nil @@ -100,14 +97,3 @@ func checkExcluded(ctx context.Context, ds internal.DataSource, fullPath, versio } return nil } - -// Paths to exclude if they match exactly. -// These are very rare, so it's simpler to hardcode them rather than use the DB. -// TODO(mkalil): Remove once go.dev/cl/761521, which provides an alternative -// implementation for case sensitive exclusions, is merged. -var caseSensitiveExcludedPaths = map[string]bool{ - "github.com/ibm/sarama": true, // https://go.dev/issue/71342 - "github.com/burntsushi/toml": true, // https://go.dev/issue/68357 - "github.com/burntSushi/toml": true, - "github.com/Burntsushi/toml": true, -} |
