Skip to content

Commit 9169c2f

Browse files
conico974Nicolas Dorseuil
andauthored
Fix site redirect resolution with length check for source paths (#3454)
Co-authored-by: Nicolas Dorseuil <[email protected]>
1 parent abe028b commit 9169c2f

File tree

1 file changed

+11
-8
lines changed
  • packages/gitbook/src/components/SitePage

1 file changed

+11
-8
lines changed

packages/gitbook/src/components/SitePage/fetch.ts

Lines changed: 11 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -71,14 +71,17 @@ async function resolvePage(context: GitBookSiteContext, params: PagePathParams |
7171
redirectPathname,
7272
]);
7373
for (const source of redirectSources) {
74-
const resolvedSiteRedirect = await getDataOrNull(
75-
context.dataFetcher.getSiteRedirectBySource({
76-
organizationId,
77-
siteId: site.id,
78-
source,
79-
siteShareKey: shareKey,
80-
})
81-
);
74+
// We try to resolve the site redirect
75+
const resolvedSiteRedirect =
76+
source.length < 512 &&
77+
(await getDataOrNull(
78+
context.dataFetcher.getSiteRedirectBySource({
79+
organizationId,
80+
siteId: site.id,
81+
source,
82+
siteShareKey: shareKey,
83+
})
84+
));
8285
if (resolvedSiteRedirect) {
8386
return redirect(linker.toLinkForContent(resolvedSiteRedirect.target));
8487
}

0 commit comments

Comments
 (0)