Coming from #2286971-83: Remove dependency of current_user on request and authentication manager →
The
PathBasedBreadcrumbBuilderis using theAccessAwareRouterto find matching routes as it is building the breadcrumb off of the current path. Unfortunately, the$requestobject passed in is incomplete (no route parameters) resulting in the exceptions as well as the auth failures.Another undesirable side effect of this is that authentication is re-run multiple times as the breadcrumb links are being built.
A way to fix this would be to changing thePathBasedBreadcrumbBuilderto use a route matcher (e.g.NestedMatcher) instead of a router to build its links.
Each time a breadcrumb is built, all route enhancers (including ParamConverters and EntityRouteEnhancers) are called for each link in the breadcrumb. This is not good for performance.
Using a request matcher directly would short circuit the calls to the route enhancers.
Active
11.0 🔥
request processing system
It affects performance. It is often combined with the Needs profiling tag.
Not all content is available!
It's likely this issue predates Contrib.social: some issue and comment data are missing.
