Cloudflare Developers•3y ago

Depends - is the website you're fetching using Cloudflare themselves?

Kkian Depends - is the website you're fetching using Cloudflare themselves?

Victor•1/18/23, 5:41 AM

About half my calls are the cloudflare api itself (for other products than workers), and the other half is not

Harrison Lo•1/18/23, 7:07 AM

Is there any way we can have build time environment variables for workers? I know there is the runtime one but that won't work in my case because I need the env in one of the imported files which is done at the top before the worker script runs

Harrison Lo•1/18/23, 7:08 AM

Looking for the equivalent of process.env.NODE_ENV in a CRA build, if that helps

kianOP•1/18/23, 7:15 AM

You can use —define to replace globals at build time with Wrangler?

Harrison Lo•1/18/23, 7:44 AM

I'm getting Uncaught ReferenceError: ENV is not definedUncaught ReferenceError: ENV is not defined Is this intended for JS files? It seems to just override the actual code, not setting the variable...

kianOP•1/18/23, 7:45 AM

If that's from trying to access

process.env

process.env

then that won't exist.

--define--define is for, let's say you had console.log(FOO)console.log(FOO) in your code - you can set it so the built output replaces the global identifier FOOFOO with "bar""bar" as opposed to a runtime variable that you'd find under the second argument to a handler.

Harrison Lo•1/18/23, 7:47 AM

I simply have WORKER_ENV === 'stage'WORKER_ENV === 'stage' in a

.ts

.ts

file

Harrison Lo•1/18/23, 7:48 AM

In my

wrangler.toml

wrangler.toml

I have define = { WORKER_ENV = "stage" }define = { WORKER_ENV = "stage" }

Harrison Lo•1/18/23, 7:49 AM

So when I build the worker, it would say "stage" is not defined, because it's treating it as a variable name, which doesn't exist

Harrison Lo•1/18/23, 7:49 AM

"stage" should be the variable value

Burrito•1/18/23, 7:52 AM

You can always fallback to using bundler features.

kianOP•1/18/23, 7:55 AM

kianOP•1/18/23, 7:58 AM

kianOP•1/18/23, 7:59 AM

it's also elided away since if (false)if (false) is dead code so esbuild removes it during minification

kianOP•1/18/23, 8:04 AM

in your scenario, if you want FOOFOO to be literally replaced with "bar""bar" then it's...

[define]
FOO = '"bar"'

[define]
FOO = '"bar"'

kianOP•1/18/23, 8:04 AM

Harrison Lo•1/18/23, 8:06 AM

Got it. Thanks for investigating, that makes sense

kianOP•1/18/23, 8:06 AM

The whole thing could do with more documentation - it has like 2-3 lines of explanation in the commands section of the docs.

It's just https://esbuild.github.io/api/#define under the hood though

esbuild - API

kianOP•1/18/23, 9:36 AM

Secrets are supposed to be managed with wrangler secret putwrangler secret put to be stored on edge, and you can use a .dev.vars.dev.vars file for local development & .gitignore.gitignore that

kianOP•1/18/23, 9:40 AM

Yes

kianOP•1/18/23, 9:46 AM

Delete it from wrangler.toml

kianOP•1/18/23, 9:46 AM

You can just leave comments on what to do

Shuoib Hossain Badon•1/18/23, 9:48 AM

I'm trying to make API in DART Language with some Dart Package. Is it possible?

kianOP•1/18/23, 9:48 AM

If it transpiles to JavaScript or targets WebAssembly, maybe?

kianOP•1/18/23, 9:48 AM

https://github.com/cloudflare/dart-worker-hello-world just transpiles it to JavaScript

GitHub

GitHub - cloudflare/dart-worker-hello-world: Dart hello world for C...

Dart hello world for Cloudflare Workers. Contribute to cloudflare/dart-worker-hello-world development by creating an account on GitHub.

Shuoib Hossain Badon•1/18/23, 9:52 AM

it gives this type of error

Shuoib Hossain Badon•1/18/23, 9:52 AM

wrangler 2.8.0
-------------------
Total Upload: 88.17 KiB / gzip: 18.06 KiB

X [ERROR] A request to the Cloudflare API (/accounts/6df991ac408d850bce9a330af5edea3e/workers/scripts/untitled_api) failed.

Uncaught Error:
at worker.js:405:13 in a
at worker.js:1158:17 in bC
at worker.js:1147:11 in ct
at worker.js:1087:18 in cx
at worker.js:2429:20 in cL
at worker.js:2394:42 in cY
at worker.js:3537:9
at worker.js:3516:9
at worker.js:3531:7 in dartProgram
at worker.js:3539:5
[code: 10021]

If you think this is a bug, please open an issue at:
https://github.com/cloudflare/wrangler2/issues/new/choose

GitHub

Build software better, together

GitHub is where people build software. More than 94 million people use GitHub to discover, fork, and contribute to over 330 million projects.

kianOP•1/18/23, 9:53 AM

It's certainly not a 'supported' scenario so you'd have to read through your generated JavaScript to see where the issue is, or ask the package author if it supports the whole JavaScript transpile step.

kianOP•1/18/23, 9:53 AM

Realistically, normal JavaScript (and TypeScript) or WebAssembly are the only Workers worth using

Original message was deleted