[smarcet]

* added content type and content length on put request to avoid 405 on ical

Author: smarcet

src/Facade/Utils/Headers.php

@@ -18,9 +18,10 @@
  */
 final class Headers
 {
-    const Depth       = 'Depth';
-    const Prefer      = 'Prefer';
-    const ContentType = 'Content-Type';
-    const IfMatch     = 'If-Match';
-    const IfNotMatch  = 'If-None-Match"';
+    const Depth         = 'Depth';
+    const Prefer        = 'Prefer';
+    const ContentType   = 'Content-Type';
+    const ContentLength = 'Content-Length';
+    const IfMatch       = 'If-Match';
+    const IfNotMatch    = 'If-None-Match"';
 }

src/Facade/Utils/HttpMethods.php

@@ -21,6 +21,7 @@
 final class HttpMethods
 {
     const Get          = 'GET';
+    const Post         = 'POST';
     const Put          = 'PUT';
     const Report       = 'REPORT';
     const PropFind     = 'PROPFIND';

src/Facade/Utils/RequestFactory.php

@@ -49,13 +49,20 @@ private static function createHeadersFor($http_method, array $params = []){
                     Headers::ContentType  => ContentTypes::Xml
                 ];
             case HttpMethods::Put:
-               $etag = $params[0];
+
+               $len  = $params[0];
+               $etag = $params[1];
+
+               $headers = [
+                   Headers::ContentLength => intval($len),
+                   Headers::ContentType   => ContentTypes::Calendar,
+               ];
+
                if(!empty($etag)){
-                   return [
-                       Headers::ContentType  => ContentTypes::Calendar,
-                       Headers::IfMatch      => $etag
-                   ];
+                   $headers[Headers::IfMatch] = $etag;
                }
+
+               return $headers;
         }
         return [];
     }
@@ -154,11 +161,28 @@ public static function createGetRequest($url){
      * @return Request
      */
      public static function createPutRequest($url, $body, $etag = null){
+
         return new Request
         (
             HttpMethods::Put,
             $url,
-            self::createHeadersFor(HttpMethods::Put, [$etag]),
+            self::createHeadersFor(HttpMethods::Put, [strlen($body), $etag]),
+            $body
+        );
+    }
+
+    /**
+     * @param string $url
+     * @param string $body
+     * @param string $etag
+     * @return Request
+     */
+    public static function createPostRequest($url, $body, $etag = null){
+        return new Request
+        (
+            HttpMethods::Post,
+            $url,
+            self::createHeadersFor(HttpMethods::Post, [$etag]),
             $body
         );
     }