fix(ApiResponse): set Accept and Content-Type headers to `applica…

…tion/json` (#50) Fixes that the API responses were missing the necessary headers. This ensures that the `Accept` and `Content-Type` headers are correctly set to `application/json` for improved compatibility and adherence to best practices in API development. Signed-off-by: Valentin Sickert <[email protected]>
RadioRoster · Dec 6, 2023 · 195a9ca · 195a9ca
1 parent 743fef8
commit 195a9ca
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/app/Http/Responses/ApiErrorResponse.php b/app/Http/Responses/ApiErrorResponse.php
@@ -18,7 +18,7 @@ public function __construct(
         private $message,
         private ?Throwable $exception = null,
         private int $status = Response::HTTP_INTERNAL_SERVER_ERROR,
-        private array $headers = []
+        private array $headers = ["Content-Type" => "application/json", "Accept" => "application/json"]
     ) {}
 
     /**

diff --git a/app/Http/Responses/ApiSuccessResponse.php b/app/Http/Responses/ApiSuccessResponse.php
@@ -16,7 +16,7 @@ class ApiSuccessResponse implements Responsable
     public function __construct(
         private $data,
         private int $status = Response::HTTP_OK,
-        private array $headers = []
+        private array $headers = ["Content-Type" => "application/json", "Accept" => "application/json"]
     ) {}
 
     /**