Skip to content
Snippets Groups Projects
Commit de136ea2 authored by Jan-Hendrik Willms's avatar Jan-Hendrik Willms
Browse files

rework JsonApiController::doesRequestHaveBody() by checking for Content-Length...

Closes #4652

Merge request studip/studip!3462
parent e39801cc
No related branches found
No related tags found
No related merge requests found
Pipeline #27421 passed
...@@ -402,7 +402,7 @@ class JsonApiController ...@@ -402,7 +402,7 @@ class JsonApiController
$request = $this->container->get('request'); $request = $this->container->get('request');
if ($this->doesRequestHaveBody($request)) { if ($this->doesRequestHaveBody($request)) {
$contentType = $request->getHeader(HeaderParametersParserInterface::HEADER_CONTENT_TYPE); $contentType = $request->getHeader(HeaderParametersParserInterface::HEADER_CONTENT_TYPE);
if (count($contentType)) { if (count($contentType) > 0) {
$mediaType = $this->factory->createMediaType( $mediaType = $this->factory->createMediaType(
MediaTypeInterface::JSON_API_TYPE, MediaTypeInterface::JSON_API_TYPE,
MediaTypeInterface::JSON_API_SUB_TYPE MediaTypeInterface::JSON_API_SUB_TYPE
...@@ -418,11 +418,6 @@ class JsonApiController ...@@ -418,11 +418,6 @@ class JsonApiController
private function doesRequestHaveBody(Request $request): bool private function doesRequestHaveBody(Request $request): bool
{ {
if (count($request->getHeader('Transfer-Encoding'))) { return in_array($request->getMethod(), ['POST', 'PATCH']);
return true;
}
$contentLength = $request->getHeader('Content-Length');
return count($contentLength) && $contentLength[0] > 0;
} }
} }
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment