Andrew Scott 3c7e637374 fix(router): Route matching should only happen once when navigating (#49163) ...

When navigating in the Router, the current approach does the redirects
and the creation of the `RouterStateSnapshot` in two separate steps
(applyRedirects and recognize). These two steps duplicate the route
matching logic, resulting in user code on routes being executing twice
(custom `UrlMatcher` and `canMatch` guards). This also duplicates the
complex matching logic in two places, which increases the bundle size
and maintenance burden.

This commit combines the `applyRedirects` and `recognize` steps into a
single matching algorithm.

fixes #26081

PR Close #49163

2023-03-29 14:40:41 -07:00

25 KiB

Raw Permalink Blame History

View Raw