Merge pull request #117 from pdsinterop/feature/multi-headers

fix for sending multiple headers with the same value

Author: Potherca

solid/lib/Controller/CalendarController.php

@@ -235,9 +235,7 @@ private function respond($response) {
 		$result = new PlainResponse($body);
 
 		foreach ($headers as $header => $values) {
-			foreach ($values as $value) {
-				$result->addHeader($header, $value);
-			}
+			$result->addHeader($header, implode(", ", $values));
 		}
 
 		$result->setStatus($statusCode);

solid/lib/Controller/ContactsController.php

@@ -236,11 +236,9 @@ private function respond($response) {
 		$result = new PlainResponse($body);
 
 		foreach ($headers as $header => $values) {
-			foreach ($values as $value) {
-				$result->addHeader($header, $value);
-			}
+			$result->addHeader($header, implode(", ", $values));
 		}
-		
+
 		$result->setStatus($statusCode);
 		return $result;
 	}

solid/lib/Controller/ProfileController.php

@@ -256,9 +256,7 @@ private function respond($response) {
 		$result = new PlainResponse($body);
 
 		foreach ($headers as $header => $values) {
-			foreach ($values as $value) {
-				$result->addHeader($header, $value);
-			}
+			$result->addHeader($header, implode(", ", $values));
 		}
 //		$origin = $_SERVER['HTTP_ORIGIN'] ?? "*";
 //		$result->addHeader('Access-Control-Allow-Credentials', 'true');

solid/lib/Controller/StorageController.php

@@ -399,10 +399,9 @@ private function respond($response) {
 		$result = new PlainResponse($body);
 
 		foreach ($headers as $header => $values) {
-			foreach ($values as $value) {
-				$result->addHeader($header, $value);
-			}
+			$result->addHeader($header, implode(", ", $values));
 		}
+
 //		$origin = $_SERVER['HTTP_ORIGIN'];
 //		$result->addHeader('Access-Control-Allow-Credentials', 'true');
 //		$result->addHeader('Access-Control-Allow-Headers', 'Content-Type, Authorization');