TASK: avoid is_array checks by using getHeaderLine()

jobee · jobee · commit f6c80644b373 · 2021-10-12T10:36:05.000+02:00
diff --git a/Classes/Http/Middleware/CheckEnabledMiddleware.php b/Classes/Http/Middleware/CheckEnabledMiddleware.php
@@ -26,9 +26,9 @@ class CheckEnabledMiddleware implements MiddlewareInterface
      */
     public function process(ServerRequestInterface $request, RequestHandlerInterface $handler): ResponseInterface
     {
-        $traceNameHeader = $request->getHeader("X-Fusion-Tracing");
-        if (is_array($traceNameHeader) ? $traceNameHeader !== [] : $traceNameHeader !== null) {
-            $this->runtimeTracing->enable(is_array($traceNameHeader) ? $traceNameHeader[0] : $traceNameHeader);
+        $traceNameHeader = $request->getHeaderLine('X-Fusion-Tracing');
+        if (!empty($traceNameHeader)) {
+            $this->runtimeTracing->enable($traceNameHeader);
         }
         return $handler->handle($request);
     }

Original file line number	Diff line number	Diff line change
`@@ -26,9 +26,9 @@ class CheckEnabledMiddleware implements MiddlewareInterface`
`26`	`26`	`*/`
`27`	`27`	`public function process(ServerRequestInterface $request, RequestHandlerInterface $handler): ResponseInterface`
`28`	`28`	`{`
`29`		`- $traceNameHeader = $request->getHeader("X-Fusion-Tracing");`
`30`		`- if (is_array($traceNameHeader) ? $traceNameHeader !== [] : $traceNameHeader !== null) {`
`31`		`- $this->runtimeTracing->enable(is_array($traceNameHeader) ? $traceNameHeader[0] : $traceNameHeader);`
	`29`	`+ $traceNameHeader = $request->getHeaderLine('X-Fusion-Tracing');`
	`30`	`+ if (!empty($traceNameHeader)) {`
	`31`	`+ $this->runtimeTracing->enable($traceNameHeader);`
`32`	`32`	`}`
`33`	`33`	`return $handler->handle($request);`
`34`	`34`	`}`