From d308d665bdb11ac52726bccb6698b3e7ab989102 Mon Sep 17 00:00:00 2001 From: k2s Date: Wed, 8 Jun 2022 18:44:46 +0000 Subject: [PATCH] fix: sed replacement of SENTRY_DSN (#2036) In log: `vikunja-frontend-1 | sed: -e expression #1, char 46: unknown option to `s'` We need to escape the VIKUNJA_SENTRY_DSN content or change sed separator to character that will not be used in sentry DNS string which I did in this commit. Signed-off-by: k2s Co-authored-by: k2s Reviewed-on: https://kolaente.dev/vikunja/frontend/pulls/2036 Reviewed-by: konrad Co-authored-by: k2s Co-committed-by: k2s --- run.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/run.sh b/run.sh index 280d5ca240..40c4bc1cf5 100755 --- a/run.sh +++ b/run.sh @@ -16,7 +16,7 @@ VIKUNJA_API_URL=$(echo $VIKUNJA_API_URL |sed 's/\//\\\//g') sed -i "s/http\:\/\/localhost\:3456//g" /usr/share/nginx/html/index.html # replacing in two steps to make sure api urls from releases are properly replaced as well sed -i "s/'\/api\/v1/'$VIKUNJA_API_URL/g" /usr/share/nginx/html/index.html sed -i "s/\.SENTRY_ENABLED = false/\.SENTRY_ENABLED = $VIKUNJA_SENTRY_ENABLED/g" /usr/share/nginx/html/index.html -sed -i "s/\.SENTRY_DSN = '.*'/\.SENTRY_DSN = '$VIKUNJA_SENTRY_DSN'/g" /usr/share/nginx/html/index.html +sed -i "s|\.SENTRY_DSN = '.*'|\.SENTRY_DSN = '$VIKUNJA_SENTRY_DSN'|g" /usr/share/nginx/html/index.html sed -i "s/listen 80/listen $VIKUNJA_HTTP_PORT/g" /etc/nginx/nginx.conf sed -i "s/listen 443/listen $VIKUNJA_HTTPS_PORT/g" /etc/nginx/nginx.conf