Hey @outsideurimagination - that create-cloudflare usage does actually create a Workers app, it's ju

Hey @outsideurimagination - that create-cloudflare usage does actually create a Workers app, it's just that the npm "start" script has not been updated to Workers (it incorrectly still mentions Pages).
But you will not that the "deploy" and "preview" scripts do actually result in workers:

    "build": "remix vite:build && wrangler pages functions build --outdir build/worker",
    "deploy": "npm run build && wrangler deploy",
    "preview": "npm run build && wrangler dev",

    "build": "remix vite:build && wrangler pages functions build --outdir build/worker",
    "deploy": "npm run build && wrangler deploy",
    "preview": "npm run build && wrangler dev",

    "build": "remix vite:build && wrangler pages functions build --outdir build/worker",
    "deploy": "npm run build && wrangler deploy",
    "preview": "npm run build && wrangler dev",

    "build": "remix vite:build && wrangler pages functions build --outdir build/worker",
    "deploy": "npm run build && wrangler deploy",
    "preview": "npm run build && wrangler dev",

It is slightly confusing because it is indeed still using the Cloudflare Pages Remix adaptor (for the moment) which generates a Pages Function output, which then needs to be converted to a Worker via the call to

wrangler pages functions build

wrangler pages functions build

wrangler pages functions build

wrangler pages functions build.

I hope to have a better template out soon

Ppetebd Hey @outsideurimagination - that create-cloudflare usage does actually create a ...

outsideurimagination•10/3/24, 12:10 PM

ok thanks. I know they’re also updating the template in the remix repo. might check that out if you haven’t to not duplicate effort https://github.com/remix-run/remix/pull/10034

GitHub

templates(cloudflare-workers): use static assets by edmundhung · Pu...

This PR updates the cloudflare-workers template to use the Static Assets feature announced today.

Note: Static assets is currently in open beta

You can learn more about Static Assets in the docum...

Foreignity•10/3/24, 1:38 PM

I'm trying to access Origin from a function, but it doesn't return anything. Is it possible to access the Origin of a request, if so, what am I doing wrong?

export async function onRequest(context) {
const origin = context.request.headers.get('Origin') || 'Not available';
...
}

berkinovish•10/3/24, 5:20 PM

Any issues with workers?

ctx

ctx

seems to be empty for us sometimes, since a couple of hours

Bberkinovish Any issues with workers? `ctx` seems to be empty for us sometimes, since a coupl...

DaniFoldi•10/3/24, 5:37 PM

I created a test worker, seems fine from a few colos that I tested

DDaniFoldi I created a test worker, seems fine from a few colos that I tested

berkinovish•10/3/24, 6:19 PM

Seems that something changed under the hood in workers - we did a check to see if ctx was a valid object somewhere in our code, and this started failing since a couple of hours. I guess a change in the latest workerd that Cloudflare started rolling out globally

DaniFoldi•10/3/24, 6:19 PM

can you share the code for this check?

DaniFoldi•10/3/24, 6:20 PM

I log the typeof ctx and the constructor.toString, and it looks correct, and identical across regions

DDaniFoldi can you share the code for this check?

berkinovish•10/3/24, 6:21 PM

We used a util that (among other things) checked for this under the hood - this is the part that failed

    for (const _ in val) {
        return true;
    }

    for (const _ in val) {
        return true;
    }

un1que1•10/3/24, 6:21 PM

Hello everyone, I need help setting up multiple Shopify stores on a single .com domain using Cloudflare. The idea is to use Workers or Transform Rules to automatically switch between different Shopify subdomains at set intervals (e.g., 12 hours per store) for load balancing. The main domain should always mask the Shopify subdomains, with all routing done through Cloudflare, without any changes on Shopify's end. Each store will have the same content. How can I achieve this and is it even possible?

Bberkinovish We used a util that (among other things) checked for this under the hood - this ...

DaniFoldi•10/3/24, 6:32 PM

could you share a little more so I can repro and escalate if it is actually a runtime change?

louiek22•10/3/24, 8:43 PM

Has anyone else seen a significant jump in bundle size when using a compatibility date >=

2024-09-23

2024-09-23

with the

nodejs_compat

nodejs_compat

flag? We saw about a 2x increase in bundle size on our workers.

kian•10/3/24, 8:46 PM

That's intentional -

nodejs_compat

nodejs_compat

implies

nodejs_compat_v2

nodejs_compat_v2

after that

compatibility_date

compatibility_date

& Wrangler adds polyfills with unenv

DDaniFoldi could you share a little more so I can repro and escalate if it is actually a ru...

berkinovish•10/3/24, 8:55 PM

Yes, this crashes with

"wrangler": "3.78.10"

"wrangler": "3.78.10"

, but works fine with wrangler on

3.78.9

3.78.9

function isNotEmptyObject<T extends Record<string, any>>(val: T | unknown): val is T {
    if (Object.prototype.toString.call(val) !== '[object Object]') return false;
    for (const _ in val as Record<string, any>) {
        return true;
    }
    return false;
}

export default {
    async fetch(request, env, ctx): Promise<Response> {
        if (!isNotEmptyObject(ctx)) throw new Error('ctx');
        return new Response('Hello World!');
    },
} satisfies ExportedHandler<Env>;

function isNotEmptyObject<T extends Record<string, any>>(val: T | unknown): val is T {
    if (Object.prototype.toString.call(val) !== '[object Object]') return false;
    for (const _ in val as Record<string, any>) {
        return true;
    }
    return false;
}

export default {
    async fetch(request, env, ctx): Promise<Response> {
        if (!isNotEmptyObject(ctx)) throw new Error('ctx');
        return new Response('Hello World!');
    },
} satisfies ExportedHandler<Env>;

We think this might be the commit that caused the behaviour change: https://github.com/cloudflare/workerd/pull/2801/files

I assume impact will be low though, pretty niche to do this check on

ctx

ctx

Llouiek22 Has anyone else seen a significant jump in bundle size when using a compatibilit...

DaniFoldi•10/3/24, 8:56 PM

that's because workers with nodejs_compat and a compat date that is newer than your date have some js polyfills added, in addition to the runtime compat mode that it enables

Bberkinovish Yes, this crashes with `"wrangler": "3.78.10"`, but works fine with wrangler on ...

DaniFoldi•10/3/24, 8:56 PM

thanks, I'll forward this info

Kkian That's intentional - `nodejs_compat` implies `nodejs_compat_v2` after that `comp...

louiek22•10/3/24, 8:58 PM

Gotcha, I thought unenv was supposed to be smart about adding polyfills. Thanks for the info!

DDaniFoldi thanks, I'll forward this info

berkinovish•10/3/24, 8:58 PM

Even though we fixed our wrangler at 3.78.6, we were still impacted - it looked like there was a gradual rollout of a newer version throughout the day

Llouiek22 Gotcha, I thought unenv was supposed to be smart about adding polyfills. Thanks ...

DaniFoldi•10/3/24, 8:58 PM

it should only add what you might use

Bberkinovish Even though we fixed our wrangler at 3.78.6, we were still impacted - it looked ...

DaniFoldi•10/3/24, 8:59 PM

correct, that's usually how runtime releases are done, and your version of wrangler only affects your local runtime version

DDaniFoldi correct, that's usually how runtime releases are done, and your version of wrang...

berkinovish•10/3/24, 8:59 PM

makes sense

DDaniFoldi it should only add what you _might_ use

louiek22•10/3/24, 8:59 PM

I'm not using anything in the worker itself, just have it for vitest.

louiek22•10/3/24, 9:01 PM

I'm guessing that extra size is from boilerplate from unenv, or maybe it's coming from my tests?

rufi•10/3/24, 11:59 PM

how many requests can a single worker handle? in AWS, GCloud and al., we see the concept of "quotas" and number of "container instances" spawned over time. We don't have such information on Cloudflare dashboard. Is it using a single worker for each request; or rather a single worker can handle millions or more requests by itself?

Walshy•10/4/24, 12:14 AM

There is no limit, I've done over 100k rps myself from my own laptop and of course we do much more as a whole platform

Each Worker can handle tons and there could many many thousands of worker instances.

Workers are light and so can be created everywhere and handle a lot

Akash•10/4/24, 1:14 AM

does snippets support kv? If not is there any plan to add this support?

Ashish Rawat•10/4/24, 5:28 AM

I wish workers has support for firebase sdk and mongodb. That's the only thing stopping me to deploy to workers. Currently have to use google cloud run for docker instance.

AAshish Rawat I wish workers has support for firebase sdk and mongodb. That's the only thing s...

hfcRed•10/4/24, 5:50 AM

Why not D1 or KV

AAkash does snippets support kv? If not is there any plan to add this support?

Walshy•10/4/24, 8:47 AM

Snippets have no bindings access. They're meant for lightweight use cases

w3bcode•10/4/24, 9:03 AM

Anyone getting error with wrangler deploy command, I am latest version 3.79.0

workers.api.error.unknown [code: 10013]

Ww3bcode Anyone getting error with wrangler deploy command, I am latest version 3.79.0 > ...

Walshy•10/4/24, 9:34 AM

What's your account ID?

w3bcode•10/4/24, 1:31 PM

9c508e2e1d1c6d075d774819cf38176c

I was using it with Cloudflare account token template settings by exposing it to environment but getting api errors, so I logged in with wrangler login and deployed. It worked that way

Focux•10/4/24, 1:35 PM

is there any template for running remix spa mode in workers?

HHello, I’m Allie!Afaik currently you would have to apply headers via a rule(or via the Worker its...

Focux•10/4/24, 1:43 PM

If the _headers file doesn't work anymore, would be good to remove it from the template, otherwise, it's a bit confusing.

FFocux If the _headers file doesn't work anymore, would be good to remove it from the t...

Isaac McFadyen•10/4/24, 1:51 PM

It still works with Pages which is what the template is designed for AFAIK

Ww3bcode 9c508e2e1d1c6d075d774819cf38176c I was using it with Cloudflare account token t...

texan•10/4/24, 1:52 PM

Hey w3b, we discovered a bug related to account owned tokens this morning. The team is looking into it, thanks!

Focux•10/4/24, 1:56 PM

oh, i thought that

pnpm create cloudflare@latest web --framework=remix --experimental

pnpm create cloudflare@latest web --framework=remix --experimental

bootstrapped a workers project

FFocux oh, i thought that `pnpm create cloudflare@latest web --framework=remix --experi...

Isaac McFadyen•10/4/24, 2:23 PM

Apologies - you might be right

Focux•10/4/24, 2:37 PM

haha no problem

kaelys.js•10/4/24, 5:05 PM

Is there a way in a worker (js) to get the current version/deployment id?

Kkaelys.js Is there a way in a worker (js) to get the current version/deployment id?

Chaika•10/4/24, 5:08 PM

https://developers.cloudflare.com/workers/runtime-apis/bindings/version-metadata/

Cloudflare Docs

Version metadata | Cloudflare Workers docs

Exposes Worker version metadata (

versionID

versionID

and

versionTag

versionTag

). These fields can be added to events emitted from the Worker to send to downstream observability systems.

HhfcRed Why not D1 or KV

Ashish Rawat•10/4/24, 5:21 PM

KV not good for everything and can't migrate. D1 is limited, no read replica, no sharding. Firebase I'm using only for firebase auth. And I'm using more node.js features (packages) that is not supported by even latest node_compat.

CChaika https://developers.cloudflare.com/workers/runtime-apis/bindings/version-metadata...

kaelys.js•10/4/24, 6:25 PM

How do I define that in the toml across multiple env? It fails when trying to add the key to separate env and warns when it’s not in separate env. Latest wrangler

Kkaelys.js How do I define that in the toml across multiple env? It fails when trying to ad...

texan•10/4/24, 6:28 PM

Can you share your toml? I set it globally as defined in the docs and not getting any errors or warnings

Ttexan Can you share your toml? I set it globally as defined in the docs and not gettin...

kaelys.js•10/4/24, 6:32 PM

Here you go, redacted some info

wrangler.toml2.17KB

kaelys.js•10/4/24, 6:33 PM

Screenshot_2024-10-04_at_11.33.24_AM.png

Walshy•10/4/24, 6:34 PM

[env.local.version_metadata]

function isNotEmptyObject<T extends Record<string, any>>(val: T | unknown): val is T { if (Object.prototype.toString.call(val) !== '[object Object]') return false; for (const _ in val as Record<string, any>) { return true; } return false; } export default { async fetch(request, env, ctx): Promise<Response> { if (!isNotEmptyObject(ctx)) throw new Error('ctx'); return new Response('Hello World!'); }, } satisfies ExportedHandler<Env>;

Hey @outsideurimagination - that create-cloudflare usage does actually create a Workers app, it's ju

Similar Threads

Similar Threads

Similar Threads