Skip to content

Commit 0a86416

Browse files
committed
fix(docker): stop injecting secure=true into CLICKHOUSE_URL
1 parent e64b101 commit 0a86416

File tree

1 file changed

+2
-11
lines changed

1 file changed

+2
-11
lines changed

docker/scripts/entrypoint.sh

Lines changed: 2 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -18,17 +18,8 @@ if [ -n "$CLICKHOUSE_URL" ] && [ "$SKIP_CLICKHOUSE_MIGRATIONS" != "1" ]; then
1818
echo "Running ClickHouse migrations..."
1919
export GOOSE_DRIVER=clickhouse
2020

21-
# Ensure secure=true is in the connection string
22-
if echo "$CLICKHOUSE_URL" | grep -q "secure="; then
23-
# secure parameter already exists, use as is
24-
export GOOSE_DBSTRING="$CLICKHOUSE_URL"
25-
elif echo "$CLICKHOUSE_URL" | grep -q "?"; then
26-
# URL has query parameters, append secure=true
27-
export GOOSE_DBSTRING="${CLICKHOUSE_URL}&secure=true"
28-
else
29-
# URL has no query parameters, add secure=true
30-
export GOOSE_DBSTRING="${CLICKHOUSE_URL}?secure=true"
31-
fi
21+
# Use the provided ClickHouse URL directly
22+
export GOOSE_DBSTRING="$CLICKHOUSE_URL"
3223

3324
export GOOSE_MIGRATION_DIR=/triggerdotdev/internal-packages/clickhouse/schema
3425
/usr/local/bin/goose up

0 commit comments

Comments
 (0)