From c5d9038945f5c8e2dd189cbf3c82536c8399f3ab Mon Sep 17 00:00:00 2001 From: Alan Agius Date: Wed, 22 Jan 2025 09:33:58 +0000 Subject: [PATCH] refactor(@angular/ssr): update `getPathSegments` to use `stripTrailingSlash` instead of `filter(Boolean)` This change modifies the `getPathSegments` function to use `stripTrailingSlash`, providing a more consistent and reliable way to handle trailing slashes in paths. This update also resolves issues causing CI failures. (cherry picked from commit b0c2d1900f3b086d13917d1187947e511f3114a3) --- packages/angular/ssr/src/routes/route-tree.ts | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/packages/angular/ssr/src/routes/route-tree.ts b/packages/angular/ssr/src/routes/route-tree.ts index 7ae0a4a84bca..e592bcb7aac9 100644 --- a/packages/angular/ssr/src/routes/route-tree.ts +++ b/packages/angular/ssr/src/routes/route-tree.ts @@ -6,7 +6,7 @@ * found in the LICENSE file at https://angular.dev/license */ -import { addLeadingSlash } from '../utils/url'; +import { addLeadingSlash, stripTrailingSlash } from '../utils/url'; import { RenderMode } from './route-config'; /** @@ -230,7 +230,7 @@ export class RouteTree = {}> * @returns An array of path segments. */ private getPathSegments(route: string): string[] { - return route.split('/').filter(Boolean); + return stripTrailingSlash(route).split('/'); } /**