Commit 0bd22da
committed
fix(@angular/ssr): replace all route parameters when resolving relative redirects
When resolving a relative `redirectTo` property for a route with multiple path parameters (e.g. `:param1/:param2`), only the first parameter segment was being converted to `*` because a non-global regular expression was used.
This commit updates `resolveRedirectTo` to use `URL_PARAMETER_GLOBAL_REGEXP` so that all parameter placeholders in the route path are properly replaced with `*`.
Fixes #335041 parent 34d558c commit 0bd22daCopy full SHA for 0bd22da
2 files changed
+30-1Lines changed: 30 additions & 1 deletion
File tree
Expand file treeCollapse file tree
Open diff view settings
Filter options
- packages/angular/ssr
- src/routes
- test/routes
Expand file treeCollapse file tree
Open diff view settings
Collapse file
packages/angular/ssr/src/routes/ng-routes.ts
Copy file name to clipboardExpand all lines: packages/angular/ssr/src/routes/ng-routes.ts+1-1Lines changed: 1 addition & 1 deletion
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
547 | 547 | |
548 | 548 | |
549 | 549 | |
550 | | - |
| 550 | + |
551 | 551 | |
552 | 552 | |
553 | 553 | |
|
Collapse file
packages/angular/ssr/test/routes/ng-routes_spec.ts
Copy file name to clipboardExpand all lines: packages/angular/ssr/test/routes/ng-routes_spec.ts+29Lines changed: 29 additions & 0 deletions
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
502 | 502 | |
503 | 503 | |
504 | 504 | |
| 505 | + |
| 506 | + |
| 507 | + |
| 508 | + |
| 509 | + |
| 510 | + |
| 511 | + |
| 512 | + |
| 513 | + |
| 514 | + |
| 515 | + |
| 516 | + |
| 517 | + |
| 518 | + |
| 519 | + |
| 520 | + |
| 521 | + |
| 522 | + |
| 523 | + |
| 524 | + |
| 525 | + |
| 526 | + |
| 527 | + |
| 528 | + |
| 529 | + |
| 530 | + |
| 531 | + |
| 532 | + |
| 533 | + |
505 | 534 | |
506 | 535 | |
507 | 536 | |
|
0 commit comments