Improve error message when route filters fail

Improve routing filter error message by indicating which routing filter
failed and the original message.

Fixes #13040

src/Routing/Router.php

@@ -18,7 +18,10 @@ use Cake\Core\Configure;
 use Cake\Http\ServerRequest;
 use Cake\Routing\Exception\MissingRouteException;
 use Cake\Utility\Inflector;
+use Exception;
 use Psr\Http\Message\ServerRequestInterface;
+use RuntimeException;
+use Throwable;
 
 /**
  * Parses the request URL into controller, action, and parameters. Uses the connected routes
@@ -585,8 +588,19 @@ class Router
     protected static function _applyUrlFilters($url)
     {
         $request = static::getRequest(true);
-        foreach (static::$_urlFilters as $filter) {
-            $url = $filter($url, $request);
+        $e = null;
+        foreach (static::$_urlFilters as $index => $filter) {
+            try {
+                $url = $filter($url, $request);
+            } catch (Exception $e) {
+                // fall through
+            } catch (Throwable $e) {
+                // fall through
+            }
+            if ($e !== null) {
+                $message = 'URL filter at index %s could not be applied. The filter failed with: %s';
+                throw new RuntimeException(sprintf($message, $index, $e->getMessage()), $e->getCode(), $e);
+            }
         }
 
         return $url;

tests/TestCase/Routing/RouterTest.php

@@ -23,6 +23,7 @@ use Cake\Routing\RouteCollection;
 use Cake\Routing\Router;
 use Cake\Routing\Route\Route;
 use Cake\TestSuite\TestCase;
+use RuntimeException;
 
 /**
  * RouterTest class
@@ -1339,6 +1340,32 @@ class RouterTest extends TestCase
     }
 
     /**
+     * Test that url filter failure gives better errors
+     *
+     * @return void
+     */
+    public function testUrlGenerationWithUrlFilterFailure()
+    {
+        $this->expectException(RuntimeException::class);
+        $this->expectExceptionMessage('URL filter at index 0 could not be applied. The filter failed with: nope');
+        Router::connect('/:lang/:controller/:action/*');
+        $request = new ServerRequest([
+            'params' => [
+                'plugin' => null,
+                'lang' => 'en',
+                'controller' => 'posts',
+                'action' => 'index'
+            ]
+        ]);
+        Router::pushRequest($request);
+
+        Router::addUrlFilter(function ($url, $request) {
+            throw new RuntimeException('nope');
+        });
+        Router::url(['controller' => 'posts', 'action' => 'index', 'lang' => 'en']);
+    }
+
+    /**
      * Test url param persistence.
      *
      * @return void