Commitad21dbe

committed

[HttpClient] Fix Content-Length header when possible

1 parent5c67f40 commitad21dbeCopy full SHA for ad21dbe

File tree

+20

-0

lines changed

+20

-0

lines changed

Lines changed: 5 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -86,6 +86,11 @@ private static function prepareRequest(?string $method, ?string $url, array $opt`
`86`	`86`
`87`	`87`	`if (isset($options['body'])) {`
`88`	`88`	`$options['body'] =self::normalizeBody($options['body']);`
	`89`	`+`
	`90`	`+if (\is_string($options['body']) && (string)\strlen($options['body']) !==substr($h =$options['normalized_headers']['content-length'][0] ??'',16)) {`
	`91`	`+$options['normalized_headers']['content-length'][0] =substr_replace($h,\strlen($options['body']),16);`
	`92`	`+$options['headers'] =array_merge(...array_values($options['normalized_headers']) ?: [[]]);`
	`93`	`+ }`
`89`	`94`	`}`
`90`	`95`
`91`	`96`	`if (isset($options['peer_fingerprint'])) {`

Lines changed: 15 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -180,6 +180,21 @@ public function testDebugInfoOnDestruct()`
`180`	`180`	`$this->assertNotEmpty($traceInfo['debug']);`
`181`	`181`	`}`
`182`	`182`
	`183`	`+publicfunctiontestFixContentLength()`
	`184`	`+ {`
	`185`	`+$client =$this->getHttpClient(__FUNCTION__);`
	`186`	`+`
	`187`	`+$response =$client->request('POST','http://localhost:8057/post', [`
	`188`	`+'body' =>'abc=def',`
	`189`	`+'headers' => ['Content-Length: 4'],`
	`190`	`+`
	`191`	`+ ]);`
	`192`	`+`
	`193`	`+$body =$response->toArray();`
	`194`	`+`
	`195`	`+$this->assertSame(['abc' =>'def','REQUEST_METHOD' =>'POST'],$body);`
	`196`	`+ }`
	`197`	`+`
`183`	`198`	`publicfunctiontestNegativeTimeout()`
`184`	`199`	`{`
`185`	`200`	`$client =$this->getHttpClient(__FUNCTION__);`

Comments

(0)