core: Ignore Tus-Resumable header in OPTIONS request (#295)

ankitpokhrel · Oct 31, 2020 · 85775de · 85775de
1 parent 1244385
commit 85775de
Show file tree

Hide file tree

Showing 2 changed files with 119 additions and 68 deletions.
diff --git a/src/Tus/Server.php b/src/Tus/Server.php
@@ -17,6 +17,7 @@
 use TusPhp\Exception\ConnectionException;
 use TusPhp\Exception\OutOfRangeException;
 use Symfony\Component\HttpFoundation\BinaryFileResponse;
+use Symfony\Component\HttpFoundation\Request as HttpRequest;
 use Symfony\Component\HttpFoundation\Response as HttpResponse;
 
 class Server extends AbstractTus
@@ -264,7 +265,7 @@ public function serve()
 
         $clientVersion = $this->getRequest()->header('Tus-Resumable');
 
-        if ($clientVersion && $clientVersion !== self::TUS_PROTOCOL_VERSION) {
+        if (HttpRequest::METHOD_OPTIONS !== $requestMethod && $clientVersion && self::TUS_PROTOCOL_VERSION !== $clientVersion) {
             return $this->response->send(null, HttpResponse::HTTP_PRECONDITION_FAILED, [
                 'Tus-Version' => self::TUS_PROTOCOL_VERSION,
             ]);