From f14ac64f2e8446844a99f639727c3dd3feae125c Mon Sep 17 00:00:00 2001 From: Gabriele Mambrini Date: Mon, 8 Nov 2021 18:06:53 +0100 Subject: [PATCH] Use posix normaliser for URL paths Signed-off-by: Gabriele Mambrini --- packages/backend-common/src/reading/FetchUrlReader.ts | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/packages/backend-common/src/reading/FetchUrlReader.ts b/packages/backend-common/src/reading/FetchUrlReader.ts index 1ae9677a85..19bf5fb140 100644 --- a/packages/backend-common/src/reading/FetchUrlReader.ts +++ b/packages/backend-common/src/reading/FetchUrlReader.ts @@ -24,7 +24,7 @@ import { SearchResponse, UrlReader, } from './types'; -import { normalize as normalizePath } from 'path'; +import path from 'path'; /** * A UrlReader that does a plain fetch of the URL. @@ -52,8 +52,10 @@ export class FetchUrlReader implements UrlReader { const paths = allowConfig.getOptionalStringArray('paths'); const checkPath = paths ? (url: URL) => { - const targetPath = normalizePath(url.pathname); - return paths.some(path => targetPath.startsWith(path)); + const targetPath = path.posix.normalize(url.pathname); + return paths.some(allowedPath => + targetPath.startsWith(allowedPath), + ); } : (_url: URL) => true; const host = allowConfig.getString('host');