From c54c4a828114a82cb53efc55128bc2edf6a49641 Mon Sep 17 00:00:00 2001 From: gersmann Date: Fri, 27 Oct 2023 14:15:15 +0200 Subject: [PATCH 1/3] fix: hide content-disposition header on /static for Safari Theres a bug in Sentry, which prevens Safari from rendering assets (JS). https://github.com/getsentry/self-hosted/issues/2285 It was fixed for the self-hosted deploy by adding a 'proxy_hide_header' statement. https://github.com/getsentry/self-hosted/commit/ab9dbbd41a31cac5b106fea24a12ba23a0e165fa --- sentry/templates/configmap-nginx.yaml | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/sentry/templates/configmap-nginx.yaml b/sentry/templates/configmap-nginx.yaml index e87e2d39d..8a0e9e820 100644 --- a/sentry/templates/configmap-nginx.yaml +++ b/sentry/templates/configmap-nginx.yaml @@ -28,7 +28,7 @@ data: location ~ ^/api/[1-9]\d*/ { proxy_pass http://relay; - } + } {{ if and .Values.nginx.metrics.enabled .Values.nginx.metrics.serviceMonitor.enabled -}} location = /status/ { @@ -40,5 +40,9 @@ data: location / { proxy_pass http://sentry; } + + location /_static/ { + proxy_hide_header Content-Disposition; + } } {{- end }} From c329c155d3b8d3d80a9774b4677b6afe72499a0e Mon Sep 17 00:00:00 2001 From: gersmann Date: Fri, 27 Oct 2023 14:16:35 +0200 Subject: [PATCH 2/3] fix: tabs / whitespace --- sentry/templates/configmap-nginx.yaml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/sentry/templates/configmap-nginx.yaml b/sentry/templates/configmap-nginx.yaml index 8a0e9e820..6cb5520a1 100644 --- a/sentry/templates/configmap-nginx.yaml +++ b/sentry/templates/configmap-nginx.yaml @@ -42,7 +42,7 @@ data: } location /_static/ { - proxy_hide_header Content-Disposition; - } + proxy_hide_header Content-Disposition; + } } {{- end }} From 43becd598c32e8d670e604944986a5741ab923f1 Mon Sep 17 00:00:00 2001 From: gersmann Date: Mon, 30 Oct 2023 11:27:49 +0100 Subject: [PATCH 3/3] chore: update chart version --- sentry/Chart.yaml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sentry/Chart.yaml b/sentry/Chart.yaml index 9ae94f53c..91981b6a7 100644 --- a/sentry/Chart.yaml +++ b/sentry/Chart.yaml @@ -2,7 +2,7 @@ apiVersion: v2 name: sentry description: A Helm chart for Kubernetes type: application -version: 20.8.1 +version: 20.8.2 appVersion: 23.10.1 dependencies: - name: memcached