No. When uploading a Worker, it will initialize it once to make sure it can start up. Here, you see

HHello, I’m Allie!No. When uploading a Worker, it will initialize it once to make sure it can star...

L

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

ah, makes sense.
So it should work after adding it with wrangler?

Llocalλorse ah, makes sense. So it should work after adding it with wrangler?

H

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

It should, yes

L

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

ok thanks!

L

localλorse•11/26/23, 8:09 AM

Btw, is Preview = staging environment?

Llocalλorse Btw, is Preview = staging environment?

H

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

wrangler preview?

L

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

I mean on cloudflare it has preview and production deployments

Llocalλorse I mean on cloudflare it has preview and production deployments

H

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

Workers Environments are disabled at the moment. Pages does have them tho

HHello, I’m Allie!Workers Environments are disabled at the moment. Pages does have them tho

L

localλorse•11/26/23, 8:13 AM

and is Preview = staging?

Llocalλorse and is Preview = staging?

H

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

On Pages, Preview is everything that isn’t production

HHello, I’m Allie!On Pages, Preview is everything that isn’t production

L

localλorse•11/26/23, 8:27 AM

ok thanks.
Btw, I had to disable automatic deployment for my project because my project was generated from a template that uses GH actions to deploy.
But that means, the normal URL to access my deployed website is not working, i.e. main.<project>.pages.dev isn't working, only some hash as subdomain (instead of "main"), which changes with every deployment.
How can I make the URL work even when automatic deployment is disabled?
So that that domain always points to the latest preview deployment?

Llocalλorse ok thanks. Btw, I had to disable automatic deployment for my project because my ...

H

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

Disabling GitHub CI deployments shouldn’t disable the domain. Do you see any deployments to the main branch?

HHello, I’m Allie!Disabling GitHub CI deployments shouldn’t disable the domain. Do you see any dep...

L

localλorse•11/26/23, 8:36 AM

I use master

Llocalλorse I use master

H

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

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λorse•11/26/23, 8:37 AM

ah!

L

localλorse•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!OP•11/26/23, 8:38 AM

Can you screenshot the deployments?

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

L

localλorse•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!OP•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!OP•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!OP•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λ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!OP•11/26/23, 9:41 AM

+200 ms

H

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

Though that is rare

H

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

Mostly Australia

Llocalλorse neither of the domains listed there work

H

Hello, I’m Allie!OP•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λ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

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

Afaik, process.env is only defined within Next

H

Hello, I’m Allie!OP•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!OP•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!OP•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!OP•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!OP•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!OP•11/26/23, 10:09 AM

Oh wait

H

Hello, I’m Allie!OP•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!OP•11/26/23, 10:12 AM

Actually, when are you calling that?

H

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

In a server route? Globally?

No. When uploading a Worker, it will initialize it once to make sure it can start up. Here, you see

Similar Threads

Similar Threads

Similar Threads