diff --git a/packages/server/src/utils/builders/railpack.ts b/packages/server/src/utils/builders/railpack.ts index 481dc0faa..991720f3a 100644 --- a/packages/server/src/utils/builders/railpack.ts +++ b/packages/server/src/utils/builders/railpack.ts @@ -2,7 +2,10 @@ import { createHash } from "node:crypto"; import type { WriteStream } from "node:fs"; import { nanoid } from "nanoid"; import type { ApplicationNested } from "."; -import { prepareEnvironmentVariables } from "../docker/utils"; +import { + parseEnvironmentKeyValuePair, + prepareEnvironmentVariables, +} from "../docker/utils"; import { getBuildAppDirectory } from "../filesystem/directory"; import { execAsync } from "../process/execAsync"; import { spawnAsync } from "../process/spawnAsync"; @@ -81,10 +84,10 @@ export const buildRailpack = async ( // Add secrets properly formatted const env: { [key: string]: string } = {}; - for (const envVar of envVariables) { - const [key, value] = envVar.split("="); + for (const pair of envVariables) { + const [key, value] = parseEnvironmentKeyValuePair(pair); if (key && value) { - buildArgs.push("--secret", `id=${key},env='${key}'`); + buildArgs.push("--secret", `id=${key},env=${key}`); env[key] = value; } } @@ -161,11 +164,11 @@ export const getRailpackCommand = ( // Add secrets properly formatted const exportEnvs = []; - for (const envVar of envVariables) { - const [key, value] = envVar.split("="); + for (const pair of envVariables) { + const [key, value] = parseEnvironmentKeyValuePair(pair); if (key && value) { - buildArgs.push("--secret", `id=${key},env='${key}'`); - exportEnvs.push(`export ${key}=${value}`); + buildArgs.push("--secret", `id=${key},env=${key}`); + exportEnvs.push(`export ${key}='${value}'`); } } diff --git a/packages/server/src/utils/docker/utils.ts b/packages/server/src/utils/docker/utils.ts index 840e6f417..61f71381e 100644 --- a/packages/server/src/utils/docker/utils.ts +++ b/packages/server/src/utils/docker/utils.ts @@ -279,6 +279,17 @@ export const prepareEnvironmentVariables = ( return resolvedVars; }; +export const parseEnvironmentKeyValuePair = ( + pair: string, +): [string, string] => { + const [key, ...valueParts] = pair.split("="); + if (!key || !valueParts.length) { + throw new Error(`Invalid environment variable pair: ${pair}`); + } + + return [key, valueParts.join("")]; +}; + export const getEnviromentVariablesObject = ( input: string | null, projectEnv?: string | null, @@ -288,7 +299,7 @@ export const getEnviromentVariablesObject = ( const jsonObject: Record = {}; for (const pair of envs) { - const [key, value] = pair.split("="); + const [key, value] = parseEnvironmentKeyValuePair(pair); if (key && value) { jsonObject[key] = value; }