- Cloudflare Developers

NNehal Ali Click to see attachment

M

Max (@rozenmd)•9/6/23, 1:09 PM

What's DB?

N

Nehal AliOP•9/6/23, 1:19 PM

DB is the variable name passed in the constructor

NNehal Ali i am getting an error "env.DB.prepare(...).run is not a function" , i logged env...

M

Matt Silverlock•9/6/23, 3:40 PM

i am getting an error "env.DB.prepare(...).run is not a function" , i logged env.DB and it is undefined,

Right —

env.DB

env.DB

is undefined because you called it

TEMPDB

TEMPDB

TEMPDB

TEMPDB. How do you get from the D1 binding on

env.TEMPDB

env.TEMPDB

env.TEMPDB

env.TEMPDB to your own class?

Can I suggest simplifying your code here? Call the methods on

env.TEMPDB

env.TEMPDB

env.TEMPDB

env.TEMPDB directly vs. abstracting them - get it working first.

S

schwartzmj•9/6/23, 4:35 PM

This works in local dev, and my other CRUD routes work without issue, however the following code throws error

Error: D1_ERROR: Error: not authorized

Error: D1_ERROR: Error: not authorized

Error: D1_ERROR: Error: not authorized

Error: D1_ERROR: Error: not authorized in production.

Anybody have any ideas? Is something not supported in D1 like

returning()

returning()

returning()

returning() or transactions or something? I'm using Miniflare in dev and it works there.

(using Drizzle ORM)

        const newTeam = await locals.db
            .transaction(async (tx) => {
                const newTeam = await tx.insert(teams).values(form.data).returning().get();
                await tx.insert(usersToTeams).values({
                    userId,
                    teamId: newTeam.id,
                    isAdmin: 1
                });
                return newTeam;
            })
            .catch((e) => {
                console.log('ERROR: ', e);
                throw error(500, 'Something went wrong creating your team.');
            });

        const newTeam = await locals.db
            .transaction(async (tx) => {
                const newTeam = await tx.insert(teams).values(form.data).returning().get();
                await tx.insert(usersToTeams).values({
                    userId,
                    teamId: newTeam.id,
                    isAdmin: 1
                });
                return newTeam;
            })
            .catch((e) => {
                console.log('ERROR: ', e);
                throw error(500, 'Something went wrong creating your team.');
            });

        const newTeam = await locals.db
            .transaction(async (tx) => {
                const newTeam = await tx.insert(teams).values(form.data).returning().get();
                await tx.insert(usersToTeams).values({
                    userId,
                    teamId: newTeam.id,
                    isAdmin: 1
                });
                return newTeam;
            })
            .catch((e) => {
                console.log('ERROR: ', e);
                throw error(500, 'Something went wrong creating your team.');
            });

        const newTeam = await locals.db
            .transaction(async (tx) => {
                const newTeam = await tx.insert(teams).values(form.data).returning().get();
                await tx.insert(usersToTeams).values({
                    userId,
                    teamId: newTeam.id,
                    isAdmin: 1
                });
                return newTeam;
            })
            .catch((e) => {
                console.log('ERROR: ', e);
                throw error(500, 'Something went wrong creating your team.');
            });

JJamie What causes `wrangler d1 execute` to print out the information after "Loading DB...

I

itsmatteomanf•9/7/23, 12:13 AM

That's cause by the API its using. The Worker binding by default return results, success and meta properties.

I

itsmatteomanf•9/7/23, 12:13 AM

https://developers.cloudflare.com/d1/platform/client-api/#return-object

I

itsmatteomanf•9/7/23, 12:14 AM

Those

***

***

***

*** are definitely GitHub Action logging issues.

JJamie Thanks I see. What causes those logs to be printed out? They don't get printed o...

I

itsmatteomanf•9/7/23, 6:50 AM

That I’m not sure… it should return something, though. If my memory is correct.

MMatt Silverlock > i am getting an error "env.DB.prepare(...).run is not a function" , **i logged...

N

Nehal AliOP•9/7/23, 8:05 AM

i tried that and it gave the same error , i also logged env.TEMPDB in the router function directly and it gave me undefined.
i don't know why it is not reading this variable , but when i use wrangler execute d1 for the commands on the database it is working fine.

NNehal Ali i tried that and it gave the same error , i also logged env.TEMPDB in the router...

M

Matt Silverlock•9/7/23, 10:11 AM

Please open a GitHub issue. There’s not enough info to go off here. Provide your code, wrangler.toml, and the commands you’ve used.

J

Jadugar_Jaggu•9/7/23, 4:18 PM

hey guys I have almost per monthly KV write operations around 5 million, and it is costing me almost 30$ + a month, My question is the D1 database in open alpha right now, can I use it in my service to replace KV for some operations, Like I am storing users IP address currently in KV , it can handle millions read and write?

J

Jadugar_Jaggu•9/7/23, 4:19 PM

I want to lower the cost of my services, and I was checking our D1 database it has very low cost compare to KV

JJadugar_Jaggu hey guys I have almost per monthly KV write operations around 5 million, and it ...

U

Unsmart•9/7/23, 4:25 PM

Its not recommended to use for production critical applications, but #durable-objects are a good alternative to KV that's cheaper.

JJadugar_Jaggu hey guys I have almost per monthly KV write operations around 5 million, and it ...

I

itsmatteomanf•9/7/23, 5:34 PM

What @Unsmart | Tech debt said is valid, but do note that D1 it can handle millions of write per minute easily. I have done it, consistently for hours and days on end.

I

itsmatteomanf•9/7/23, 5:34 PM

(While still reading fast, and read replicas aren’t even here yet)

JJamie If I redirect the output to a file then I see the results locally, but don't see...

I

itsmatteomanf•9/8/23, 6:43 AM

No, I don’t think I understood here… which output should be turned off? If you write it doesn’t return anything, apart from meta and success.

JJamie If I redirect the output to a file then I see the results locally, but don't see...

M

Max (@rozenmd)•9/8/23, 8:56 AM

run wrangler with the

WRANGLER_LOG

WRANGLER_LOG

WRANGLER_LOG

WRANGLER_LOG env variable set to

error

error

or

none

none

none

none

VViper What my `.env` file should be like? I have it like this. ```.env binding="DB"...

M

Matt Silverlock•9/8/23, 11:20 AM

Nothing to do with your

.env

.env

file. You need to create the Pages binding to D1 in the dashboard first: https://developers.cloudflare.com/pages/platform/functions/bindings/#d1-databases

Bindings · Cloudflare Pages docs

A binding enables your Pages Functions to interact with resources on the Cloudflare developer platform. Use bindings to integrate your Pages Functions …

VViper What my `.env` file should be like? I have it like this. ```.env binding="DB"...

M

Matt Silverlock•9/8/23, 11:20 AM

process.env

process.env

is how they are injected into Next.js within your code.

VViper It works fine in production now but, I was wondering if there's a way to test it...

J

James•9/8/23, 11:43 AM

If you want to use your bindings in

next dev

next dev

, I would suggest using a library like

cf-bindings-proxy

cf-bindings-proxy

cf-bindings-proxy

cf-bindings-proxy

JJames If you want to use your bindings in `next dev`, I would suggest using a library ...

U

Unsmart•9/8/23, 2:55 PM

Speaking of this was just bouta ask... you ever run into an issue where deployed works perfect as expected but locally it will say the command worked but it did nothing?

Not sure if this is the proxy somehow like caching a response or if its a miniflare issue

S

slowpoke•9/8/23, 8:05 PM

Baby goo-goo-ga-ga question if I want to create a table with a key that is an array or strings, is that possible / what is the syntax?

CREATE TABLE IF NOT EXISTS 'profiles' (
    'full_name' TEXT,
    'fields_of_study' Array of Text?
);

CREATE TABLE IF NOT EXISTS 'profiles' (
    'full_name' TEXT,
    'fields_of_study' Array of Text?
);

CREATE TABLE IF NOT EXISTS 'profiles' (
    'full_name' TEXT,
    'fields_of_study' Array of Text?
);

CREATE TABLE IF NOT EXISTS 'profiles' (
    'full_name' TEXT,
    'fields_of_study' Array of Text?
);

K

kian•9/8/23, 8:07 PM

I don't think there's any native support for arrays, but you could store it as a JSON array & query it as such - https://developers.cloudflare.com/d1/learning/querying-json/

Query JSON · Cloudflare D1 docs

D1 has built-in support for querying and parsing JSON data stored within a database. This enables you to:

S

slowpoke•9/8/23, 8:17 PM

Using

    'fields_of_study' TEXT ARRAY[4],

'fields_of_study' TEXT ARRAY[4],

    'fields_of_study' TEXT ARRAY[4],

'fields_of_study' TEXT ARRAY[4], seems to do something, although when running a

.prepare

.prepare

.prepare

.prepare query to insert / retrieve data I'm not sure if it's working how I think it should.

Thanks though I'll look into that. :)

S

slowpoke•9/8/23, 8:41 PM

Ok so I'm unsure if I'm understanding this documentation correctly, but it seems there are

json

json

json

json helper functions for use with your SQL queries i.e.

SELECT * ...

SELECT * ...

SELECT * ...

SELECT * ... or

db.prepare("SELECT * ...

db.prepare("SELECT * ...

db.prepare("SELECT * ...

db.prepare("SELECT * ...

I'm just trying to understand how to put it into practice -
I have

const data = db.prepare("SELECT * FROM profiles WHERE user_id = ?").bind(id).first()`

const data = db.prepare("SELECT * FROM profiles WHERE user_id = ?").bind(id).first()`

const data = db.prepare("SELECT * FROM profiles WHERE user_id = ?").bind(id).first()`

const data = db.prepare("SELECT * FROM profiles WHERE user_id = ?").bind(id).first()`

This data is already just about what I want, except my

fields_of_study

fields_of_study

fields_of_study

fields_of_study key is the string

"['Major 1', 'Major 2']"

"['Major 1', 'Major 2']"

"['Major 1', 'Major 2']"

"['Major 1', 'Major 2']".

Am I storing the data incorrectly in the column or do I need to restructure my query so that D1 will parse the array into a javascript object?

Sslowpoke Ok so I'm unsure if I'm understanding this documentation correctly, but it seems...

I

itsmatteomanf•9/8/23, 8:43 PM

I'm sure you already read this, but there are examples here

https://developers.cloudflare.com/d1/learning/querying-json/

I

itsmatteomanf•9/8/23, 8:43 PM

What's the question, though? The return value?

I

itsmatteomanf•9/8/23, 8:43 PM

The JSON helper functions help for traversing or fetching specific parts of a JSON object stored in a column.

I

itsmatteomanf•9/8/23, 8:43 PM

They run at the SQL query level, not in the Worker.

S

slowpoke•9/8/23, 8:44 PM

I just thought though, I think I could always just do JSON.parse() on the key before returning it to the client

I

itsmatteomanf•9/8/23, 8:44 PM

Yeah, you can lul

S

slowpoke•9/8/23, 8:44 PM

I think I thought that this was used to also parse JSON strings without needing the use of parse. Lmao

S

slowpoke•9/8/23, 8:45 PM

I see what you mean. This will help you traverse your JSON strings in a column without needing to like… do everything that entails to update one object key.

I

itsmatteomanf•9/8/23, 8:45 PM

Exactly, but also allows you to create virtual columns from a JSON object, avoiding the work of iterating through data or returning more than necessary.

I

itsmatteomanf•9/8/23, 8:46 PM

Or do sort of multi-query with a single query using the

json_each()

json_each()

json_each()

json_each()

S

slowpoke•9/8/23, 8:47 PM

Yep, tysm! Sorry about me overthinking it. thonk

I

itsmatteomanf•9/8/23, 8:52 PM

No prob

Similar Threads