Merge pull request #1 from XDAM/bugfix/remove-content-length-header

Content-Length header may be sent in proxy CONNECT
jumbojett · May 11, 2020 · a770de3 · a770de3
2 parents 103cd52 + 8136235
commit a770de3
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -14,7 +14,7 @@ and this project adheres to [Semantic Versioning](http://semver.org/).
 * Getters for public parameters #204
 
 ### Removed
-*
+* Removed explicit content-length header - caused issues with proxy servers
 
 
 ## [0.9.0]

diff --git a/src/OpenIDConnectClient.php b/src/OpenIDConnectClient.php
@@ -1085,7 +1085,6 @@ protected function fetchURL($url, $post_body = null, $headers = array()) {
 
             // Add POST-specific headers
             $headers[] = "Content-Type: {$content_type}";
-            $headers[] = 'Content-Length: ' . strlen($post_body);
 
         }
-Original file line number
+Diff line change
@@ Expand Up @@
                 // Add POST-specific headers
                 $headers[] = "Content-Type: {$content_type}";
-                $headers[] = 'Content-Length: ' . strlen($post_body);
             }
@@ Expand Down @@