Ah! I just set the env var with wrangler and tried to deploy again but it failed again with, even th

Ah!
I just set the env var with wrangler and tried to deploy again but it failed again with, even though the env var is present in the worker named

t4-api

t4-api

✘ [ERROR] A request to the Cloudflare API (/accounts/<account>/workers/scripts/t4-api) failed.

  Uncaught Error: The OPENAI_API_KEY environment variable is missing or empty; either provide it, or instantiate the OpenAI client with an apiKey option, like new OpenAI({ apiKey: 'My API Key' }).
    at worker.js:12490:13 in OpenAI
    at worker.js:12579:14
   [code: 10021]

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


The process '/home/runner/.bun/bin/bunx' failed with exit code 1
🚨 Action failed

✘ [ERROR] A request to the Cloudflare API (/accounts/<account>/workers/scripts/t4-api) failed.

  Uncaught Error: The OPENAI_API_KEY environment variable is missing or empty; either provide it, or instantiate the OpenAI client with an apiKey option, like new OpenAI({ apiKey: 'My API Key' }).
    at worker.js:12490:13 in OpenAI
    at worker.js:12579:14
   [code: 10021]

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


The process '/home/runner/.bun/bin/bunx' failed with exit code 1
🚨 Action failed

✘ [ERROR] A request to the Cloudflare API (/accounts/<account>/workers/scripts/t4-api) failed.

  Uncaught Error: The OPENAI_API_KEY environment variable is missing or empty; either provide it, or instantiate the OpenAI client with an apiKey option, like new OpenAI({ apiKey: 'My API Key' }).
    at worker.js:12490:13 in OpenAI
    at worker.js:12579:14
   [code: 10021]

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


The process '/home/runner/.bun/bin/bunx' failed with exit code 1
🚨 Action failed

✘ [ERROR] A request to the Cloudflare API (/accounts/<account>/workers/scripts/t4-api) failed.

  Uncaught Error: The OPENAI_API_KEY environment variable is missing or empty; either provide it, or instantiate the OpenAI client with an apiKey option, like new OpenAI({ apiKey: 'My API Key' }).
    at worker.js:12490:13 in OpenAI
    at worker.js:12579:14
   [code: 10021]

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


The process '/home/runner/.bun/bin/bunx' failed with exit code 1
🚨 Action failed

Llocalλorse Ah! I just set the env var with wrangler and tried to deploy again but it failed...

Hello, I’m Allie!•11/26/23, 10:01 AM

Oh wait… it probably expects a global. What happens if you initialize it as it asks, in the constructor?

HHello, I’m Allie!Oh wait… it probably expects a global. What happens if you initialize it as it a...

localλorseOP•11/26/23, 10:01 AM

I am!

const apiKey = process.env.OPENAI_API_KEY
const organization = process.env.OPENAI_ORG_ID
const openai = new OpenAI({ apiKey, organization })

const apiKey = process.env.OPENAI_API_KEY
const organization = process.env.OPENAI_ORG_ID
const openai = new OpenAI({ apiKey, organization })

localλorseOP•11/26/23, 10:02 AM

btw, this constructor would otherwise read the same env var if I didn't pass this:

  constructor({
    apiKey = Core.readEnv('OPENAI_API_KEY'),
    organization = Core.readEnv('OPENAI_ORG_ID') ?? null,
    ...opts
  }: ClientOptions = {}) {

  constructor({
    apiKey = Core.readEnv('OPENAI_API_KEY'),
    organization = Core.readEnv('OPENAI_ORG_ID') ?? null,
    ...opts
  }: ClientOptions = {}) {

Llocalλorse btw, this constructor would otherwise read the same env var if I didn't pass thi...

Hello, I’m Allie!•11/26/23, 10:06 AM

Afaik, process.env is only defined within Next

Hello, I’m Allie!•11/26/23, 10:06 AM

At least on Workers

HHello, I’m Allie!Afaik, process.env is only defined within Next

localλorseOP•11/26/23, 10:06 AM

what do you mean by that?

localλorseOP•11/26/23, 10:07 AM

this project uses next.js

Llocalλorse what do you mean by that?

Hello, I’m Allie!•11/26/23, 10:07 AM

Workers don’t have global env variables. Next gets around this somehow, but afaik that only applies within the Next scope

Hello, I’m Allie!•11/26/23, 10:08 AM

So a 3rd-party package wouldn’t be able to read any variables

localλorseOP•11/26/23, 10:08 AM

what's the Next scope?

Llocalλorse what's the Next scope?

Hello, I’m Allie!•11/26/23, 10:08 AM

The code you write with Next

HHello, I’m Allie!So a 3rd-party package wouldn’t be able to read any variables

localλorseOP•11/26/23, 10:08 AM

but the way I'm reading from process.env and passing it to the constructor should work, right?

Llocalλorse but the way I'm reading from process.env and passing it to the constructor shoul...

Hello, I’m Allie!•11/26/23, 10:09 AM

Yes. Just to be sure, check whether they are defined, by console-logging them

Hello, I’m Allie!•11/26/23, 10:09 AM

Oh wait

Hello, I’m Allie!•11/26/23, 10:09 AM

Do deployments actually show logs other than errors…

Llocalλorse but the way I'm reading from process.env and passing it to the constructor shoul...

Hello, I’m Allie!•11/26/23, 10:12 AM

Actually, when are you calling that?

Hello, I’m Allie!•11/26/23, 10:12 AM

In a server route? Globally?

HHello, I’m Allie!In a server route? Globally?

localλorseOP•11/26/23, 10:14 AM

I define

const apiKey

const apiKey

as a global var in a ts file under packages/api/src/routes

Llocalλorse I define `const apiKey` as a global var in a ts file under packages/api/src/rout...

Hello, I’m Allie!•11/26/23, 10:15 AM

That’s probably the issue then. Environment Variables aren’t populated on startup

localλorseOP•11/26/23, 10:15 AM

Hello, I’m Allie!•11/26/23, 10:15 AM

So you can’t start the API client globally

HHello, I’m Allie!That’s probably the issue then. Environment Variables aren’t populated on startu...

localλorseOP•11/26/23, 10:15 AM

oh. so when are they populated?

Llocalλorse oh. so when are they populated?

Hello, I’m Allie!•11/26/23, 10:15 AM

Once the Worker/Function is actually invoked

HHello, I’m Allie!Once the Worker/Function is actually invoked

localλorseOP•11/26/23, 10:16 AM

so I can't cache the return value of the OpenAI constructor between invocations?

localλorseOP•11/26/23, 10:16 AM

(in a global var or otherwise)

Llocalλorse so I can't cache the return value of the OpenAI constructor between invocations?

Hello, I’m Allie!•11/26/23, 10:17 AM

You could, but you can’t actually construct it until you have received a request

localλorseOP•11/26/23, 10:18 AM

ok, so what would you recommend for doing that? some kind of nullable memoization type

Llocalλorse ok, so what would you recommend for doing that? some kind of nullable memoizatio...

Hello, I’m Allie!•11/26/23, 10:19 AM

Just have a variable that can be an OpenAI API client, but by default is null. Then, whenever you need it, use the variable if present, otherwise initialize it first

HHello, I’m Allie!Just have a variable that *can* be an OpenAI API client, but by default is null....

localλorseOP•11/26/23, 10:19 AM

ah ok, thanks :)

�

🎄pog troll🔔🎄•11/26/23, 2:20 PM

@Helpflare cloudflare keeps on saying failure

fengyangsi•11/26/23, 4:02 PM

hi, I code a worker and it can solve POST request but I get 403

fengyangsi•11/26/23, 4:03 PM

Why the respond is 403?

fengyangsi•11/26/23, 4:04 PM

rn_image_picker_lib_temp_4a397703-060a-4134-8bf6-1c00910026de.jpg

fengyangsi•11/26/23, 4:05 PM

And 421

fengyangsi•11/26/23, 4:05 PM

But my program haven't return 403 or 421

Ffengyangsi hi, I code a worker and it can solve POST request but I get 403

Isaac McFadyen•11/26/23, 5:06 PM

Those kind of error pages usually appear if your browser is making a request that isn't valid according to the HTTP specification.

Isaac McFadyen•11/26/23, 5:06 PM

For example, if you send a body in a GET request then you might get that. Or if you send a header name that isn't valid. Or any number of other things really.

Isaac McFadyen•11/26/23, 5:06 PM

Take a look and make sure that the POST request you're making is valid and looks fine syntactically

scotto•11/26/23, 6:07 PM

There is any good project example for workers and D1 to start from ? Like with multiples APIs / files

dave•11/26/23, 7:47 PM

Question, for those of you who have had their Workers DDoS'd, how was the experience? Does CF do a decent job of blocking requests before they hit the Workers?

dave•11/26/23, 7:51 PM

nice. I was debating if I should use Google Cloud Run without Workers, but I think it makes sense to always use Workers in front.

dave•11/26/23, 7:51 PM

it does increased the cost from $0.20/million requests to $0.50/million requests when stacking a Worker on top, but the benefit of DDoS protection makes it nice.

Original message was deleted

zegevlier•11/26/23, 8:04 PM

You can even block by header value on, so if they did have a valid authentication token somehow they would still get blocked.

meow•11/26/23, 9:29 PM

Is middleware only available for Cloudflare Pages and not Workers?

James•11/26/23, 9:41 PM

Middleware is a thing for Functions since they have file-based routing, and the concept of Middleware makes a lot of sense there for handling auth, etc.

James•11/26/23, 9:41 PM

Many popular routers and frameworks for Workers also support Middleware though, such as https://hono.dev/

Fun Miles•11/27/23, 12:12 AM

My cron worker job got killed for exceeding time (32,000 ms). Is there a way to increase that number?

Fun Miles•11/27/23, 12:15 AM

The docs, under CPU time say

When the schedule interval is more than 1 hour, a scheduled Worker may run for up to 15 minutes.

When the schedule interval is more than 1 hour, a scheduled Worker may run for up to 15 minutes.

. My job runs once per month.

Shane•11/27/23, 12:43 AM

wrangler 3.17.1
-------------------
⎔ Starting local server...
✘ [ERROR] service core:user:search: Uncaught TypeError: Incorrect type for map entry 'DEFAULT_PAGE_SIZE': the provided value is not of type 'function or ExportedHandler'.

Is there a way to get a more detailed error here? I removed default_page_size from my code entirely and I'm still getting the error for some reason

Shane•11/27/23, 12:47 AM

odd, no error when I turn off local mode

Ah! I just set the env var with wrangler and tried to deploy again but it failed again with, even th

Similar Threads

Similar Threads

Similar Threads