- 🇷🇴Romania amateescu
Found this old issue while investigating lots of alias lookups on a large site with many patches and contrib modules :/ I've also read all of #2480811: Cache incoming path processing and route matching → , and this bit from the current issue summary stood out:
Make sure the cache key only gets set after routing has completed just in case.
I think the "just in case" part should actually be strictly enforced, because at the moment it's super easy to completely disable core's alias preloading optimizations by running outbound path processing before routing is done. I've found three examples in just one site, who knows how many other cases are out there in the wild...
- (proposed core patch) 🐛 Aliased paths cannot be set as front page Needs review
- (contrib - Force Password Change → ) Starting from #3108636: Infinite redirect loop when used in combination with drupal/rename_admin_paths → , should hopefully be fixed in 🐛 Generated URL language prefix mismatch causes infinite redirect loop RTBC
- (contrib - Redirect → ) Starting from #2704213: "Redirect from non-canonical URLs to the canonical URLs" not working with language code in url → https://git.drupalcode.org/project/redirect/-/commit/d1eaf7c7a9d12fdd496...