I use master

H

The subdomain on Pages is based on the branch you are on. If you don’t have a main branch, you won’t have a main subdomain

L

localλorseOP•11/26/23, 8:37 AM

ah!

L

localλorseOP•11/26/23, 8:37 AM

but it doesn't show me any URL with master subdomain

Llocalλorse but it doesn't show me any URL with master subdomain

H

Hello, I’m Allie!•11/26/23, 8:38 AM

Can you screenshot the deployments?

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

L

localλorseOP•11/26/23, 8:41 AM

I'm on phone right now, but I just tried the URL:
master.<project>.pages.dev doesn't work, only main.<project>.pages.dev but that shows an old outdated deployment, not the latest!
and I never had a main branch btw

P

Patie•11/26/23, 8:51 AM

its possible to write into Durable Object from external server by API?

PPatie its possible to write into Durable Object from external server by API?

H

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

Yes, but the call still has to go through a Worker

HHello, I’m Allie!Yes, but the call still has to go through a Worker

P

Patie•11/26/23, 9:29 AM

thank you, how long take DO read and write, i know there are many factors, byt typically

PPatie thank you, how long take DO read and write, i know there are many factors, byt t...

H

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

From the external API, or within the DO?

HHello, I’m Allie!From the external API, or within the DO?

P

Patie•11/26/23, 9:35 AM

from worker

PPatie from worker

H

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

That would be dependent on where the DO is. If it is nearby, I’d say ~10-20 ms

HHello, I’m Allie!That would be dependent on where the DO is. If it is nearby, I’d say ~10-20 ms

P

Patie•11/26/23, 9:37 AM

and if its not nearby?

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

L

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

L

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

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

L

localλorseOP•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

Patie•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λorseOP•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

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

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

  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λorseOP•11/26/23, 10:06 AM

what do you mean by that?

L

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

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

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λorseOP•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λorseOP•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λorseOP•11/26/23, 10:16 AM

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

L

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?

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

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λorseOP•11/26/23, 10:19 AM

ah ok, thanks :)

Similar Threads

Similar Threads

Similar Threads