Skip to content

Commit 2a02c87

Browse files
committed
chore: simplify trailing slash handling
1 parent a0db05f commit 2a02c87

File tree

3 files changed

+1
-68
lines changed

3 files changed

+1
-68
lines changed

packages/runtime/src/helpers/utils.ts

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -52,13 +52,6 @@ export const toNetlifyRoute = (nextRoute: string): Array<string> => {
5252
netlifyRoutes.unshift(netlifyRoute)
5353
}
5454

55-
// RSC data routes get an extra route with `/index.rsc` so that the router
56-
// can match a different URL with and without a trailing slash
57-
if (nextRoute.endsWith('.rsc')) {
58-
const rscIndexRoute = nextRoute.replace(/\.rsc$/, '/index.rsc')
59-
netlifyRoutes.push(rscIndexRoute)
60-
}
61-
6255
return netlifyRoutes.map((route) =>
6356
route
6457
// Replace catch-all, e.g., [...slug]

packages/runtime/src/templates/edge-shared/rsc-data.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ const noop = () => {}
2727
// Ensure that routes with and without a trailing slash map to different ODB paths
2828
const rscifyPath = (route: string) => {
2929
if (route.endsWith('/')) {
30-
return route + 'index.rsc'
30+
return route.slice(0, -1) + '.rsc/'
3131
}
3232
return route + '.rsc'
3333
}

test/__snapshots__/index.spec.js.snap

Lines changed: 0 additions & 60 deletions
Original file line numberDiff line numberDiff line change
@@ -1359,18 +1359,6 @@ Array [
13591359
"status": 200,
13601360
"to": "/.netlify/builders/___netlify-odb-handler",
13611361
},
1362-
Object {
1363-
"force": false,
1364-
"from": "/blog/:author/:slug/index.rsc",
1365-
"status": 200,
1366-
"to": "/.netlify/builders/___netlify-odb-handler",
1367-
},
1368-
Object {
1369-
"force": false,
1370-
"from": "/blog/:author/index.rsc",
1371-
"status": 200,
1372-
"to": "/.netlify/builders/___netlify-odb-handler",
1373-
},
13741362
Object {
13751363
"force": true,
13761364
"from": "/blog/erica",
@@ -1383,12 +1371,6 @@ Array [
13831371
"status": 200,
13841372
"to": "/.netlify/builders/___netlify-odb-handler",
13851373
},
1386-
Object {
1387-
"force": true,
1388-
"from": "/blog/erica/index.rsc",
1389-
"status": 200,
1390-
"to": "/.netlify/builders/___netlify-odb-handler",
1391-
},
13921374
Object {
13931375
"force": true,
13941376
"from": "/blog/nick",
@@ -1401,12 +1383,6 @@ Array [
14011383
"status": 200,
14021384
"to": "/.netlify/builders/___netlify-odb-handler",
14031385
},
1404-
Object {
1405-
"force": true,
1406-
"from": "/blog/nick/index.rsc",
1407-
"status": 200,
1408-
"to": "/.netlify/builders/___netlify-odb-handler",
1409-
},
14101386
Object {
14111387
"force": true,
14121388
"from": "/blog/rob",
@@ -1419,12 +1395,6 @@ Array [
14191395
"status": 200,
14201396
"to": "/.netlify/builders/___netlify-odb-handler",
14211397
},
1422-
Object {
1423-
"force": true,
1424-
"from": "/blog/rob/index.rsc",
1425-
"status": 200,
1426-
"to": "/.netlify/builders/___netlify-odb-handler",
1427-
},
14281398
Object {
14291399
"force": true,
14301400
"from": "/blog/sarah",
@@ -1437,12 +1407,6 @@ Array [
14371407
"status": 200,
14381408
"to": "/.netlify/builders/___netlify-odb-handler",
14391409
},
1440-
Object {
1441-
"force": true,
1442-
"from": "/blog/sarah/index.rsc",
1443-
"status": 200,
1444-
"to": "/.netlify/builders/___netlify-odb-handler",
1445-
},
14461410
Object {
14471411
"force": false,
14481412
"from": "/broken-image",
@@ -1521,18 +1485,6 @@ Array [
15211485
"status": 200,
15221486
"to": "/.netlify/builders/___netlify-odb-handler",
15231487
},
1524-
Object {
1525-
"force": false,
1526-
"from": "/es/blog/:author/:slug/index.rsc",
1527-
"status": 200,
1528-
"to": "/.netlify/builders/___netlify-odb-handler",
1529-
},
1530-
Object {
1531-
"force": false,
1532-
"from": "/es/blog/:author/index.rsc",
1533-
"status": 200,
1534-
"to": "/.netlify/builders/___netlify-odb-handler",
1535-
},
15361488
Object {
15371489
"force": false,
15381490
"from": "/es/broken-image",
@@ -1761,18 +1713,6 @@ Array [
17611713
"status": 200,
17621714
"to": "/.netlify/builders/___netlify-odb-handler",
17631715
},
1764-
Object {
1765-
"force": false,
1766-
"from": "/fr/blog/:author/:slug/index.rsc",
1767-
"status": 200,
1768-
"to": "/.netlify/builders/___netlify-odb-handler",
1769-
},
1770-
Object {
1771-
"force": false,
1772-
"from": "/fr/blog/:author/index.rsc",
1773-
"status": 200,
1774-
"to": "/.netlify/builders/___netlify-odb-handler",
1775-
},
17761716
Object {
17771717
"force": false,
17781718
"from": "/fr/broken-image",

0 commit comments

Comments
 (0)