fix(settings): Fix log file download return type

Signed-off-by: provokateurin <kate@provokateurin.de>

Author: provokateurin

apps/settings/lib/Controller/LogSettingsController.php

@@ -48,21 +48,21 @@ public function __construct(string $appName, IRequest $request, Log $logger) {
 	 *
 	 * @NoCSRFRequired
 	 *
-	 * @psalm-suppress MoreSpecificReturnType The value of Content-Disposition is not relevant
-	 * @psalm-suppress LessSpecificReturnStatement The value of Content-Disposition is not relevant
-	 * @return StreamResponse<Http::STATUS_OK, array{Content-Type: 'application/octet-stream', 'Content-Disposition': string}>
+	 * @return StreamResponse<Http::STATUS_OK, array{Content-Type: 'application/octet-stream', 'Content-Disposition': 'attachment; filename="nextcloud.log"'}>
 	 *
 	 * 200: Logfile returned
 	 */
 	public function download() {
 		if (!$this->log instanceof Log) {
 			throw new \UnexpectedValueException('Log file not available');
 		}
-		$resp = new StreamResponse($this->log->getLogPath());
-		$resp->setHeaders([
-			'Content-Type' => 'application/octet-stream',
-			'Content-Disposition' => 'attachment; filename="nextcloud.log"',
-		]);
-		return $resp;
+		return new StreamResponse(
+			$this->log->getLogPath(),
+			Http::STATUS_OK,
+			[
+				'Content-Type' => 'application/octet-stream',
+				'Content-Disposition' => 'attachment; filename="nextcloud.log"',
+			],
+		);
 	}
 }

apps/settings/openapi.json

@@ -44,7 +44,10 @@
                         "headers": {
                             "Content-Disposition": {
                                 "schema": {
-                                    "type": "string"
+                                    "type": "string",
+                                    "enum": [
+                                        "attachment; filename=\"nextcloud.log\""
+                                    ]
                                 }
                             }
                         },