diff --git a/.changeset/light-jars-cheat.md b/.changeset/light-jars-cheat.md new file mode 100644 index 000000000000..c286810b4a22 --- /dev/null +++ b/.changeset/light-jars-cheat.md @@ -0,0 +1,5 @@ +--- +'astro': patch +--- + +Handle inlining non-string boolean environment variables diff --git a/.changeset/spicy-pugs-wonder.md b/.changeset/spicy-pugs-wonder.md new file mode 100644 index 000000000000..568538810eb8 --- /dev/null +++ b/.changeset/spicy-pugs-wonder.md @@ -0,0 +1,5 @@ +--- +'@astrojs/cloudflare': patch +--- + +Fix runtime env var handling diff --git a/packages/astro/src/vite-plugin-env/index.ts b/packages/astro/src/vite-plugin-env/index.ts index 8d5a9c1c3df8..75a37d693b85 100644 --- a/packages/astro/src/vite-plugin-env/index.ts +++ b/packages/astro/src/vite-plugin-env/index.ts @@ -31,7 +31,11 @@ function getPrivateEnv( // Ignore public env var if (envPrefixes.every((prefix) => !key.startsWith(prefix))) { if (typeof process.env[key] !== 'undefined') { - const value = process.env[key]; + let value = process.env[key]; + // Replacements are always strings, so try to convert to strings here first + if (typeof value !== 'string') { + value = `${value}`; + } // Boolean values should be inlined to support `export const prerender` // We already know that these are NOT sensitive values, so inlining is safe if (value === '0' || value === '1' || value === 'true' || value === 'false') { diff --git a/packages/integrations/cloudflare/src/index.ts b/packages/integrations/cloudflare/src/index.ts index fa064e0ac2f6..40ee006f3ed2 100644 --- a/packages/integrations/cloudflare/src/index.ts +++ b/packages/integrations/cloudflare/src/index.ts @@ -91,6 +91,14 @@ export default function createIntegration(args?: Options): AstroIntegration { } vite.ssr ||= {}; vite.ssr.target = 'webworker'; + + // Cloudflare env is only available per request. This isn't feasible for code that access env vars + // in a global way, so we shim their access as `process.env.*`. We will populate `process.env` later + // in its fetch handler. + vite.define = { + 'process.env': 'process.env', + ...vite.define, + }; } }, 'astro:build:ssr': ({ entryPoints }) => { diff --git a/packages/integrations/cloudflare/test/basics.test.js b/packages/integrations/cloudflare/test/basics.test.js index b97079b8f134..9aa78f98e4ca 100644 --- a/packages/integrations/cloudflare/test/basics.test.js +++ b/packages/integrations/cloudflare/test/basics.test.js @@ -2,7 +2,7 @@ import { loadFixture, runCLI } from './test-utils.js'; import { expect } from 'chai'; import * as cheerio from 'cheerio'; -describe.skip('Basic app', () => { +describe('Basic app', () => { /** @type {import('./test-utils').Fixture} */ let fixture; /** @type {import('./test-utils').WranglerCLI} */