Fix issue when capturing routes with OPTIONS

rougin · Jan 12, 2024 · e3cd43a · e3cd43a
1 parent 072cb7f
commit e3cd43a
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/src/Routing/Dispatcher.php b/src/Routing/Dispatcher.php
@@ -15,7 +15,7 @@ class Dispatcher implements DispatcherInterface
     /**
      * @var string[]
      */
-    protected $allowed = array('DELETE', 'GET', 'PATCH', 'POST', 'PUT');
+    protected $allowed = array('DELETE', 'GET', 'OPTIONS', 'PATCH', 'POST', 'PUT');
 
     /**
      * @var \Rougin\Slytherin\Routing\RouterInterface
@@ -101,7 +101,7 @@ protected function validMethod($method)
     {
         if (in_array($method, $this->allowed)) return true;
 
-        $message = 'Used method is not allowed';
+        $message = 'Used method is not allowed (' . $method . ')';
 
         throw new \BadMethodCallException($message);
     }
@@ -125,7 +125,9 @@ protected function match($method, $uri)
 
             $sameMethod = $route->getMethod() === $method;
 
-            if ($matched && $sameMethod)
+            $isOptions = $route->getMethod() === 'OPTIONS';
+
+            if ($matched && ($sameMethod || $isOptions))
             {
                 return $route->setParams($matches);
             }