Skip to content

Commit 439fd60

Browse files
authored
remove redirect to simple added in #9908 (#10503)
Simple is an API endpoint, and redirects can be costly to our backends. It _also_ could lead to people relying on this undocumented URL.
1 parent aff8d87 commit 439fd60

File tree

2 files changed

+0
-2
lines changed

2 files changed

+0
-2
lines changed

tests/unit/test_routes.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -420,7 +420,6 @@ def add_policy(name, filename):
420420
pretend.call("/sponsor/", "/sponsors/", domain=warehouse),
421421
pretend.call("/u/{username}/", "/user/{username}/", domain=warehouse),
422422
pretend.call("/p/{name}/", "/project/{name}/", domain=warehouse),
423-
pretend.call("/s/{name}/", "/simple/{name}/", domain=warehouse),
424423
pretend.call("/pypi/{name}/", "/project/{name}/", domain=warehouse),
425424
pretend.call(
426425
"/pypi/{name}/{version}/", "/project/{name}/{version}/", domain=warehouse

warehouse/routes.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -347,7 +347,6 @@ def includeme(config):
347347

348348
# Legacy URLs
349349
config.add_route("legacy.api.simple.index", "/simple/", domain=warehouse)
350-
config.add_redirect("/s/{name}/", "/simple/{name}/", domain=warehouse)
351350
config.add_route(
352351
"legacy.api.simple.detail",
353352
"/simple/{name}/",

0 commit comments

Comments
 (0)