diff --git a/core/modules/system/src/PathBasedBreadcrumbBuilder.php b/core/modules/system/src/PathBasedBreadcrumbBuilder.php index 7f2c43c412810f3..adfd83acc11267d 100644 --- a/core/modules/system/src/PathBasedBreadcrumbBuilder.php +++ b/core/modules/system/src/PathBasedBreadcrumbBuilder.php @@ -20,6 +20,7 @@ use Drupal\Core\Routing\RouteMatchInterface; use Drupal\Core\Session\AccountInterface; use Drupal\Core\StringTranslation\StringTranslationTrait; use Drupal\Core\Url; +use Symfony\Component\HttpFoundation\Exception\BadRequestException; use Symfony\Component\HttpFoundation\Request; use Symfony\Component\HttpKernel\Exception\AccessDeniedHttpException; use Symfony\Component\HttpKernel\Exception\NotFoundHttpException; @@ -211,7 +212,12 @@ class PathBasedBreadcrumbBuilder implements BreadcrumbBuilderInterface { if (!empty($exclude[$path])) { return NULL; } - $request = Request::create($path); + try { + $request = Request::create($path); + } + catch (BadRequestException) { + return NULL; + } // Performance optimization: set a short accept header to reduce overhead in // AcceptHeaderMatcher when matching the request. $request->headers->set('Accept', 'text/html'); diff --git a/core/modules/system/tests/src/Unit/Breadcrumbs/PathBasedBreadcrumbBuilderTest.php b/core/modules/system/tests/src/Unit/Breadcrumbs/PathBasedBreadcrumbBuilderTest.php index 35d1615ced9b5ea..241e1a00a76a1ed 100644 --- a/core/modules/system/tests/src/Unit/Breadcrumbs/PathBasedBreadcrumbBuilderTest.php +++ b/core/modules/system/tests/src/Unit/Breadcrumbs/PathBasedBreadcrumbBuilderTest.php @@ -336,6 +336,28 @@ class PathBasedBreadcrumbBuilderTest extends UnitTestCase { $this->assertEquals(Cache::PERMANENT, $breadcrumb->getCacheMaxAge()); } + /** + * Tests the build method with an invalid path. + * + * @covers ::build + * @covers ::getRequestForPath + */ + public function testBuildWithInvalidPath(): void { + // The parse_url() function returns FALSE for '/:123/foo' so the + // Request::create() method therefore considers it to be an invalid URI. + $this->context->expects($this->once()) + ->method('getPathInfo') + ->willReturn('/:123/foo/bar'); + + $breadcrumb = $this->builder->build($this->createMock('Drupal\Core\Routing\RouteMatchInterface')); + + // No path matched, though at least the frontpage is displayed. + $this->assertEquals([0 => new Link('Home', new Url(''))], $breadcrumb->getLinks()); + $this->assertEqualsCanonicalizing(['url.path.is_front', 'url.path.parent'], $breadcrumb->getCacheContexts()); + $this->assertEqualsCanonicalizing([], $breadcrumb->getCacheTags()); + $this->assertEquals(Cache::PERMANENT, $breadcrumb->getCacheMaxAge()); + } + /** * Tests the applied method. *