From a37af4690b196fe66a271433b6b16b176ba3e3c6 Mon Sep 17 00:00:00 2001
From: Jan-Hendrik Willms <tleilax+studip@gmail.com>
Date: Wed, 14 Dec 2022 13:40:56 +0000
Subject: [PATCH] add Content-Length header for render_json(), fixes #1889
Closes #1889
Merge request studip/studip!1237
---
app/controllers/studip_controller.php | 8 +++++++-
1 file changed, 7 insertions(+), 1 deletion(-)
diff --git a/app/controllers/studip_controller.php b/app/controllers/studip_controller.php
index 0347b37ea19..1b351eafa1d 100644
--- a/app/controllers/studip_controller.php
+++ b/app/controllers/studip_controller.php
@@ -12,6 +12,9 @@
require_once 'studip_controller_properties_trait.php';
require_once 'studip_response.php';
+/**
+ * @property StudipResponse $response
+ */
abstract class StudipController extends Trails_Controller
{
use StudipControllerPropertiesTrait;
@@ -413,8 +416,11 @@ abstract class StudipController extends Trails_Controller
*/
public function render_json($data)
{
+ $json = json_encode($data);
+
$this->set_content_type('application/json;charset=utf-8');
- $this->render_text(json_encode($data));
+ $this->response->add_header('Content-Length', strlen($json));
+ $this->render_text($json);
}
/**
--
GitLab