I don't know about most people, but I've had great success using kysely: http://kysely.dev

Wwilhuff I don't know about most people, but I've had great success using kysely: http://...

W

wilhuffOP•8/24/24, 9:15 PM

It does need a dialect plugin for D1, but that's available here: https://github.com/aidenwallis/kysely-d1

GitHub

GitHub - aidenwallis/kysely-d1: D1 dialect for Kysely

D1 dialect for Kysely. Contribute to aidenwallis/kysely-d1 development by creating an account on GitHub.

.

.gwapes•8/24/24, 9:23 PM

omg that looks amazing... thanks so much

Wwilhuff I don't know about most people, but I've had great success using kysely: http://...

.

.gwapes•8/24/24, 9:33 PM

Running into a ts error here... wondering if you can help

const db = new Kysely<Database>({
    dialect: new D1Dialect({ database: env.voting }) 
});

export interface User {
    userId: string
    votes: number;
    lastVoted: number;
    votes_this_month: number;
};

export interface Database {
    voting: User;
}
No overload matches this call.
  Overload 1 of 2, '(args: KyselyConfig): Kysely<Database>', gave the following error.
    Type 'D1Dialect' is not assignable to type 'Dialect'.
      The types of 'createAdapter().acquireMigrationLock' are incompatible between these types.
        Type '(_db: Kysely<any>, _opt: MigrationLockOptions) => Promise<void>' is not assignable to type '(db: Kysely<any>, options: MigrationLockOptions) => Promise<void>'.
          Types of parameters '_db' and 'db' are incompatible.
            Type 'import("c:/Users/kyebm/OneDrive/Desktop/Desktop (1)/cf bot/node_modules/kysely/dist/esm/kysely", { with: { "resolution-mode": "import" } }).Kysely<any>' is not assignable to type 'import("c:/Users/kyebm/OneDrive/Desktop/Desktop (1)/cf bot/node_modules/kysely/dist/cjs/kysely").Kysely<any>'.
              Property '#private' in type 'Kysely' refers to a different member that cannot be accessed from within type 'Kysely'.
  Overload 2 of 2, '(args: KyselyProps): Kysely<Database>', gave the following error.
    Type 'D1Dialect' is not assignable to type 'Dialect'.ts(2769)

const db = new Kysely<Database>({
    dialect: new D1Dialect({ database: env.voting }) 
});

export interface User {
    userId: string
    votes: number;
    lastVoted: number;
    votes_this_month: number;
};

export interface Database {
    voting: User;
}
No overload matches this call.
  Overload 1 of 2, '(args: KyselyConfig): Kysely<Database>', gave the following error.
    Type 'D1Dialect' is not assignable to type 'Dialect'.
      The types of 'createAdapter().acquireMigrationLock' are incompatible between these types.
        Type '(_db: Kysely<any>, _opt: MigrationLockOptions) => Promise<void>' is not assignable to type '(db: Kysely<any>, options: MigrationLockOptions) => Promise<void>'.
          Types of parameters '_db' and 'db' are incompatible.
            Type 'import("c:/Users/kyebm/OneDrive/Desktop/Desktop (1)/cf bot/node_modules/kysely/dist/esm/kysely", { with: { "resolution-mode": "import" } }).Kysely<any>' is not assignable to type 'import("c:/Users/kyebm/OneDrive/Desktop/Desktop (1)/cf bot/node_modules/kysely/dist/cjs/kysely").Kysely<any>'.
              Property '#private' in type 'Kysely' refers to a different member that cannot be accessed from within type 'Kysely'.
  Overload 2 of 2, '(args: KyselyProps): Kysely<Database>', gave the following error.
    Type 'D1Dialect' is not assignable to type 'Dialect'.ts(2769)

const db = new Kysely<Database>({
    dialect: new D1Dialect({ database: env.voting }) 
});

export interface User {
    userId: string
    votes: number;
    lastVoted: number;
    votes_this_month: number;
};

export interface Database {
    voting: User;
}
No overload matches this call.
  Overload 1 of 2, '(args: KyselyConfig): Kysely<Database>', gave the following error.
    Type 'D1Dialect' is not assignable to type 'Dialect'.
      The types of 'createAdapter().acquireMigrationLock' are incompatible between these types.
        Type '(_db: Kysely<any>, _opt: MigrationLockOptions) => Promise<void>' is not assignable to type '(db: Kysely<any>, options: MigrationLockOptions) => Promise<void>'.
          Types of parameters '_db' and 'db' are incompatible.
            Type 'import("c:/Users/kyebm/OneDrive/Desktop/Desktop (1)/cf bot/node_modules/kysely/dist/esm/kysely", { with: { "resolution-mode": "import" } }).Kysely<any>' is not assignable to type 'import("c:/Users/kyebm/OneDrive/Desktop/Desktop (1)/cf bot/node_modules/kysely/dist/cjs/kysely").Kysely<any>'.
              Property '#private' in type 'Kysely' refers to a different member that cannot be accessed from within type 'Kysely'.
  Overload 2 of 2, '(args: KyselyProps): Kysely<Database>', gave the following error.
    Type 'D1Dialect' is not assignable to type 'Dialect'.ts(2769)

const db = new Kysely<Database>({
    dialect: new D1Dialect({ database: env.voting }) 
});

export interface User {
    userId: string
    votes: number;
    lastVoted: number;
    votes_this_month: number;
};

export interface Database {
    voting: User;
}
No overload matches this call.
  Overload 1 of 2, '(args: KyselyConfig): Kysely<Database>', gave the following error.
    Type 'D1Dialect' is not assignable to type 'Dialect'.
      The types of 'createAdapter().acquireMigrationLock' are incompatible between these types.
        Type '(_db: Kysely<any>, _opt: MigrationLockOptions) => Promise<void>' is not assignable to type '(db: Kysely<any>, options: MigrationLockOptions) => Promise<void>'.
          Types of parameters '_db' and 'db' are incompatible.
            Type 'import("c:/Users/kyebm/OneDrive/Desktop/Desktop (1)/cf bot/node_modules/kysely/dist/esm/kysely", { with: { "resolution-mode": "import" } }).Kysely<any>' is not assignable to type 'import("c:/Users/kyebm/OneDrive/Desktop/Desktop (1)/cf bot/node_modules/kysely/dist/cjs/kysely").Kysely<any>'.
              Property '#private' in type 'Kysely' refers to a different member that cannot be accessed from within type 'Kysely'.
  Overload 2 of 2, '(args: KyselyProps): Kysely<Database>', gave the following error.
    Type 'D1Dialect' is not assignable to type 'Dialect'.ts(2769)

Original message was deleted

.

.gwapes•8/25/24, 11:08 AM

That's what I switched to last night. That other one looks great, but what Patrick said is very true. Just doesn't seem to work at all

Original message was deleted

W

wilhuffOP•8/25/24, 3:09 PM

AFAIK transactions don’t exist in the D1 API, so I wouldn’t ding the dialect author for that.

Should there be an issue with the dialect, it’s so simple that it would be straightforward to just vendor the code. That’s definitely not the thing that worries me.

I started with raw SQL and it was manually lining up the INSERT INTO column names, bind parameters, and values from my data that sent me screaming.

The lack transactions in D1 forces me to make every insert include on conflict clauses. Kysely (plus a few extensions to the API to make on conflict set from excluded more concise) has cut away huge swaths of code. I won’t go back.

P

pixelbaker•8/25/24, 4:25 PM

I'm running into an odd issue with workers and routes. Have a simple worker that returns a static maintenance HTML page in JavaScript. When I add the route for the production domain (

*domain.org/*

*domain.org/*

) it still returns the origin site. Double checked that A records are proxied at Cloudflare. The weird part is that it works just fine with the staging domain (

*domain.dev/*

*domain.dev/*

). I've also double checked that there are no bulk redirects that might be getting in the way. Any ideas?

Ppixelbaker I'm running into an odd issue with workers and routes. Have a simple worker that...

P

pixelbaker•8/25/24, 4:53 PM

Figured it out. Origin webhost was WP Engine and they include Cloudflare SaaS, so it was going to their config first before hitting my own Cloudflare account.

A

Artem•8/25/24, 5:37 PM

Hi, folks!
Build Cache can be a great improvement to the thing I'm building. Is there a way to provide an explicit cache directory? If not, I highly recommend you add one (putting it into wrangler config may be a good start).

A

Artem•8/25/24, 5:37 PM

Would like to cache a Remix build, the framework itself isn't officially supported in Build Cache.

W

Walshy•8/25/24, 5:55 PM

#pages-discussions but no, no arbritrary cache dir specification - something i have requested though myself
as for having remix supported, i'd recommend posting in #pages-feature-suggestions

B

Bernardo•8/25/24, 10:09 PM

Hi everyone, my MTLS certificates are not working anymore

B

Bernardo•8/25/24, 10:09 PM

Is anyone having troubles with mtls certificates too?

P

Ping for toast•8/26/24, 1:59 AM

Timers are only available inside of the Request Context.

Does this mean I cannot use

setTimeout

setTimeout

with cron?

PPing for toast > Timers are only available inside of the Request Context. Does this mean I cann...

I

Isaac McFadyen•8/26/24, 2:05 AM

I believe request context includes cron here.

I

Isaac McFadyen•8/26/24, 2:05 AM

Just means no global scope timers.

IIsaac McFadyen Just means no global scope timers.

P

Ping for toast•8/26/24, 2:05 AM

ok good

P

Ping for toast•8/26/24, 2:05 AM

I'm trying to send emails on cron and AWS has a throttle

I

Isaac McFadyen•8/26/24, 2:06 AM

What are you using setTimeout/setInterval for?

I

Isaac McFadyen•8/26/24, 2:06 AM

Ah, OK.

IIsaac McFadyen Ah, OK.

P

Ping for toast•8/26/24, 2:06 AM

Just need to space out calls to the AWS api

I

IdkWhatever69•8/26/24, 2:26 AM

i pass the minify flag to wrangler to reduce size of my worker code
this results in minified function names in errors
is there a way to pass a source map (probably a/b problem ik) or is there any other way to "get/view" the full function names in the stack trace?

Screenshot_2024-08-25_at_10.23.33_PM.png

B

Brett Willis•8/26/24, 8:17 AM

I'm starting to get HTTP

520 Bad Gateway

520 Bad Gateway

responses from my Worker which I have not touched deployed in several days. From the logs, the requests don't event seem to reach the worker (no logs or exceptions). Some requests do reach the worker, but POST/PUT/PATCH requests do not and return 520. There is no backend and, responses are served solely by the worker itself. How do I debug this?

B

Brett Willis•8/26/24, 8:24 AM

This seems to only be an issue for requests via the

workers.dev

workers.dev

domain. Requests via a custom domain route function normally.

N

NotJones•8/26/24, 11:51 AM

does anyone know if workers are still having issues with discord hooks ?

N

NotJones•8/26/24, 11:52 AM

sometimes I'm not getting notifications sent but i know that the script ran, as it added to my KV

W

Walshy•8/26/24, 11:52 AM

can you expand?

WWalshy can you expand?

N

NotJones•8/26/24, 11:55 AM

expand ?

W

Walshy•8/26/24, 11:56 AM

what is the issue you're actually seeing? have you tried anything to debug? code? any info really that can help

W

Walshy•8/26/24, 11:56 AM

what "notifications" are these? how are they sent?

WWalshy what "notifications" are these? how are they sent?

I

IdkWhatever69•8/26/24, 11:57 AM

post req to discord api

N

NotJones•8/26/24, 11:57 AM

discord hook yes, using fetch

W

Walshy•8/26/24, 11:58 AM

ok, have you debugged what discord has returned? workers all come from the same IP so if you don't send a bot token it's pretty easy to be rate limited.

W

Walshy•8/26/24, 11:58 AM

we use webhooks extensively in here from Workers (see #cloudflare-blog & #cloudflare-status along with other internal channels) so things are running fine

N

NotJones•8/26/24, 11:58 AM

oh I see

NNotJones sometimes I'm not getting notifications sent but i know that the script ran, as ...

I

IdkWhatever69•8/26/24, 11:59 AM

iirc there 2 endpoints for discord webhooks
one returns a response based on success
the other just returns 201

I

IdkWhatever69•8/26/24, 11:59 AM

make sure u sue the first and log it some other manner

N

NotJones•8/26/24, 12:00 PM

I will try adding a bot token and see if this solves my issue

N

NotJones•8/26/24, 12:00 PM

tyvm @Walshy | Deploying and @IdkWhatever69

NNotJones I will try adding a bot token and see if this solves my issue

I

IdkWhatever69•8/26/24, 12:01 PM

follow the discord api specs
dont just randomly add headers

IIdkWhatever69 follow the discord api specs dont just randomly add headers

N

NotJones•8/26/24, 12:02 PM

Yea ofc, I will read first how the headers should look like

IIdkWhatever69 i pass the minify flag to wrangler to reduce size of my worker code this results...

I

IdkWhatever69•8/26/24, 12:02 PM

bump if anyone has suggestions

W

Walshy•8/26/24, 12:06 PM

https://developers.cloudflare.com/workers/observability/source-maps/

W

Walshy•8/26/24, 12:06 PM

alternatively, generate yourself with

wrangler deploy --dry-run --outdir dist

wrangler deploy --dry-run --outdir dist

.

.gwapes•8/26/24, 2:47 PM

Is this a bug? I literally just made the worker and used

npx create-cloudflare@latest testapp --existing-script testapp

npx create-cloudflare@latest testapp --existing-script testapp

.

.gwapes•8/26/24, 2:48 PM

It also makes the wrangler.toml main be index.ts when the script it downloads is worker.ts

.

.gwapes•8/26/24, 2:51 PM

Well actually no, it seems to all be a bit messed up

I don't know about most people, but I've had great success using kysely: http://kysely.dev

Similar Threads

Similar Threads

Similar Threads