From e3d10dbfbff89a758c54bf56f156f213aed3d412 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=C3=89milien=20Devos?= <contact@emiliendevos.be>
Date: Tue, 26 Oct 2021 14:16:46 +0000
Subject: [PATCH] don't include SENTRY_DSN in the docker image

---
 .github/workflows/docker-image.yml | 2 --
 Dockerfile.new                     | 1 +
 2 files changed, 1 insertion(+), 2 deletions(-)

diff --git a/.github/workflows/docker-image.yml b/.github/workflows/docker-image.yml
index 057a690..c0aef36 100644
--- a/.github/workflows/docker-image.yml
+++ b/.github/workflows/docker-image.yml
@@ -61,7 +61,6 @@ jobs:
         context: ./
         file: ./Dockerfile.new
         build-args: |
-            SENTRY_DSN=$SENTRY_DSN
             release=1
         platforms: linux/amd64,linux/arm64/v8
         push: true
@@ -73,7 +72,6 @@ jobs:
       with:
         context: ./
         build-args: |
-            SENTRY_DSN=$SENTRY_DSN
             release=1
         file: ./Dockerfile
         push: true
diff --git a/Dockerfile.new b/Dockerfile.new
index 7b4e801..972f530 100644
--- a/Dockerfile.new
+++ b/Dockerfile.new
@@ -4,6 +4,7 @@ RUN apk add --no-cache 'crystal=1.2.1-r0' shards sqlite-static yaml-static yaml-
 WORKDIR /invidious
 COPY ./invidious/shard.yml ./shard.yml
 COPY ./invidious/shard.lock ./shard.lock
+# Sentry is just for reporting Invidious crashes, no personal data is collected.
 RUN yq e -i '.dependencies.raven.github = "Sija/raven.cr"' shard.yml
 RUN yq e -i '.targets.sentry_crash_handler.main = "lib/raven/src/crash_handler.cr"' shard.yml
 RUN shards install