Skip to content

Commit 2a82832

Browse files
committed
fix(@angular/ssr): use wildcard server route configuration on the '/' route when the app router is empty
1 parent 4fcce7d commit 2a82832

File tree

2 files changed

+10
-8
lines changed

2 files changed

+10
-8
lines changed

packages/angular/ssr/src/routes/ng-routes.ts

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -480,12 +480,12 @@ export async function getRoutesFromAngularRouterConfig(
480480
}
481481
}
482482
} else {
483-
const renderMode = serverConfigRouteTree?.match('')?.renderMode ?? RenderMode.Prerender;
484-
485-
routesResults.push({
483+
const rootRouteMetadata = serverConfigRouteTree?.match('') ?? {
486484
route: '',
487-
renderMode,
488-
});
485+
renderMode: RenderMode.Prerender,
486+
};
487+
488+
routesResults.push(rootRouteMetadata);
489489
}
490490

491491
return {

packages/angular/ssr/test/routes/ng-routes_spec.ts

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -340,8 +340,8 @@ describe('extractRoutesAndCreateRouteTree', () => {
340340
);
341341
});
342342

343-
it('should apply RenderMode matching the wildcard when no Angular routes are defined', async () => {
344-
setAngularAppTestingManifest([], [{ path: '**', renderMode: RenderMode.Server }]);
343+
it('should use wildcard configuration when no Angular routes are defined', async () => {
344+
setAngularAppTestingManifest([], [{ path: '**', renderMode: RenderMode.Server, status: 201 }]);
345345

346346
const { errors, routeTree } = await extractRoutesAndCreateRouteTree(
347347
url,
@@ -351,6 +351,8 @@ describe('extractRoutesAndCreateRouteTree', () => {
351351
);
352352

353353
expect(errors).toHaveSize(0);
354-
expect(routeTree.toObject()).toEqual([{ route: '/', renderMode: RenderMode.Server }]);
354+
expect(routeTree.toObject()).toEqual([
355+
{ route: '/**', renderMode: RenderMode.Server, status: 201 },
356+
]);
355357
});
356358
});

0 commit comments

Comments
 (0)