and if its not nearby? 😬

HHello, I’m Allie!Can you screenshot the deployments?

L

localλorse•11/26/23, 9:39 AM

L

localλorse•11/26/23, 9:39 AM

(btw, I don't have a main branch, only master and dev)

L

localλorse•11/26/23, 9:40 AM

neither of the domains listed there work

PPatie and if its not nearby? 😬

H

Hello, I’m Allie!•11/26/23, 9:41 AM

+200 ms

H

Hello, I’m Allie!•11/26/23, 9:41 AM

Though that is rare

H

Hello, I’m Allie!•11/26/23, 9:41 AM

Mostly Australia

Llocalλorse neither of the domains listed there work

H

Hello, I’m Allie!•11/26/23, 9:42 AM

That’s because production doesn’t have a valid deployment

HHello, I’m Allie!Though that is rare

P

PatieOP•11/26/23, 9:43 AM

thats totally ok for my case if its rarely ~200ms, thank you

HHello, I’m Allie!That’s because production doesn’t have a valid deployment

L

localλorse•11/26/23, 9:57 AM

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...

H

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...

L

localλorse•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 })

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 })

L

localλorse•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 = {}) {

  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...

H

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

Afaik, process.env is only defined within Next

H

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

L

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

what do you mean by that?

L

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

this project uses next.js

Llocalλorse what do you mean by that?

H

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

H

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

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

L

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

what's the Next scope?

Llocalλorse what's the Next scope?

H

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

L

localλorse•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...

H

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

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

H

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

Oh wait

H

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...

H

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

Actually, when are you calling that?

H

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

In a server route? Globally?

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

L

localλorse•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...

H

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

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

L

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

H

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...

L

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

oh. so when are they populated?

Llocalλorse oh. so when are they populated?

H

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

L

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

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

L

localλorse•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?

H

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

L

localλorse•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...

H

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....

L

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

ah ok, thanks :)

�

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

@Helpflare cloudflare keeps on saying failure

F

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

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

F

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

Why the respond is 403?

F

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

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

F

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

And 421

F

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

I

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.

I

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.

I

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

S

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

D

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?

D

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.

Similar Threads

Similar Threads

Similar Threads