t3-oss / t3-env

https://env.t3.gg
MIT License
2.45k stars 79 forks source link

fix: identify Deno as server (#154) #220

Closed michaellopez closed 2 months ago

michaellopez commented 2 months ago

@juliusmarminge We hit the faulty server detection explained in #154 when using t3-env on Netlify in a middleware in Next.js. Netlify uses Deno to bundle for edge, and this resulted in builds failing with "❌ Attempted to access a server-side environment variable on the client".

I think Netlify + Next.js middleware + t3-env qualifies as "common setups widely used" as you put it in your comment

We would appreciate out of the box support for this combo.

vercel[bot] commented 2 months ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
t3-env ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 17, 2024 2:03pm
vercel[bot] commented 2 months ago

@michaellopez is attempting to deploy a commit to the t3-oss Team on Vercel.

A member of the Team first needs to authorize it.

juliusmarminge commented 2 months ago

Can you fix lint?

bun lint:fix

michaellopez commented 2 months ago

Can you fix lint?

bun lint:fix

@juliusmarminge Lint is fixed, sorry I missed that.