fix: test handler returns 204 by default

Author: AidasK

CHANGELOG.md

@@ -1,4 +1,5 @@
 # 1.0.0
 ## Breaking Changes
  * Flow\Exception was replaced by Flow\FileOpenException and Flow\FileLockException
- * php requirement was changed to >=5.4
+ * php requirement was changed to >=5.4
+ * if chunk was not found, 204 status is returned instead of 404

README.md

@@ -35,7 +35,7 @@ if ($_SERVER['REQUEST_METHOD'] === 'GET') {
     if ($file->checkChunk()) {
         header("HTTP/1.1 200 Ok");
     } else {
-        header("HTTP/1.1 404 Not Found");
+        header("HTTP/1.1 204 No Content");
         return ;
     }
 } else {

src/Flow/Basic.php

@@ -30,8 +30,8 @@ public static function save($destination, $config, RequestInterface $request = n
             if ($file->checkChunk()) {
                 header("HTTP/1.1 200 Ok");
             } else {
-                header("HTTP/1.1 404 Not Found");
-
+                // The 204 response MUST NOT include a message-body, and thus is always terminated by the first empty line after the header fields.
+                header("HTTP/1.1 204 No Content");
                 return false;
             }
         } else {
@@ -40,7 +40,6 @@ public static function save($destination, $config, RequestInterface $request = n
             } else {
                 // error, invalid chunk upload request, retry
                 header("HTTP/1.1 400 Bad Request");
-
                 return false;
             }
         }