Fixed LinkedIn resource owner to always put Content-Length header i…

…n POST reqeusts : cherry pick 859ec55
Wisembly · Feb 8, 2021 · 7640802 · 7640802
1 parent 74d2c7e
commit 7640802
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 5 deletions.
diff --git a/OAuth/ResourceOwner/AbstractResourceOwner.php b/OAuth/ResourceOwner/AbstractResourceOwner.php
@@ -248,10 +248,12 @@ protected function httpRequest($url, $content = null, array $headers = [], $meth
             $method = null === $content || '' === $content ? 'GET' : 'POST';
         }
 
-        $headers += array('User-Agent' => 'HWIOAuthBundle (https://github.com/hwi/HWIOAuthBundle)');
-        if (is_string($content)) {
-            $headers += array('Content-Length' => (string) strlen($content));
-        } elseif (is_array($content)) {
+        $headers += ['User-Agent' => 'HWIOAuthBundle (https://github.com/hwi/HWIOAuthBundle)'];
+        if (\is_string($content)) {
+            if (!isset($headers['Content-Length'])) {
+                $headers += ['Content-Length' => (string) \strlen($content)];
+            }
+        } elseif (\is_array($content)) {
             $content = http_build_query($content, '', '&');
         }
 

diff --git a/OAuth/ResourceOwner/LinkedinResourceOwner.php b/OAuth/ResourceOwner/LinkedinResourceOwner.php
@@ -73,10 +73,16 @@ protected function doGetTokenRequest($url, array $parameters = [])
      */
     protected function httpRequest($url, $content = null, array $headers = [], $method = null)
     {
-        // Linkedin v2 API is supposed to require Content-Type: application/json but it works without
+        // LinkedIn v2 API is supposed to require Content-Type: application/json but it works without
         // and request to get the access token doesn't seems to work with Content-Type: application/json
         // so we don't put any Content-Type header.
         // Skip the Content-Type header in GenericOAuth2ResourceOwner::httpRequest
+        //
+        // LinkedIn API requires to always set Content-Length in POST requests
+        if ('POST' === $method) {
+            $headers['Content-Length'] = \is_string($content) ? (string) \strlen($content) : '0';
+        }
+
         return AbstractResourceOwner::httpRequest($url, $content, $headers, $method);
     }