Extend next-auth session with drizzle adapter

I followed the steps to extend the next-auth session object with a custom userRole property. I extended the Session and User interface, I have the userRole property in my Users DB table and I include the role in the session callback in authOptions. The only type error that I am getting is from the adapter (DrizzleAdapter). But when I check the session object the role property is missing.

I've searched everywhere but I cannot figure it out or find a solution. Does anyone have the same problem or know a fix?

IkBenJur•11/30/23, 8:18 PM

I have never done this myself but reading the docs. Did you add the profile callback in the providers array? This is what the docs says for adding roles to users: The profile() callback's return value is used to create users in the database. That's it! Your newly created users will now have an assigned role.

If you also want to use the role on the client, you can expose it via the session callback.

IkBenJur•11/30/23, 8:18 PM

The link if you want to read the rest btw https://authjs.dev/guides/basics/role-based-access-control

Role-based access control | Auth.js

There are two ways to add role-based access control (RBAC) to your application, based on the session strategy you choose. Let's see an example for each of these.

IkBenJur•11/30/23, 8:44 PM

Actually now that I’m thinking. Did you set a default value for the role in your schema? Let’s say you create a new user in your db and you don’t set the role manually. Don’t have a default value set in the schema either and try to access the role through user.role in the session. The user.role will be empty and thus the role field will never be passed along right?

IkBenJur•11/30/23, 8:45 PM

Or am I thinking wrong

Steve MelonsOP•11/30/23, 10:51 PM

Thanks for the feedback, and yes I have done all those things. I figured it out. It is actually a bug. The next-auth Drizzle Adapter only fetches the defaults from the db

Steve MelonsOP•11/30/23, 10:52 PM

I created an issue but sadly it keeps getting closed because of the reproduction

Steve MelonsOP•11/30/23, 10:52 PM

https://github.com/nextauthjs/next-auth/issues/9296

GitHub

DrizzleAdapter overwrites custom tables with defaults · Issue #9296...

Adapter type @auth/drizzle-adapter Environment System: OS: Linux 5.15 Ubuntu 20.04.4 LTS (Focal Fossa) CPU: (32) x64 13th Gen Intel(R) Core(TM) i9-13900K Memory: 11.79 GB / 15.49 GB Container: Yes ...

AnonymousSmoothBrain•12/5/23, 2:59 AM

+1 to this bug

EnricoBaivo•12/14/23, 11:12 PM

Hello Steve Melons, did you find a solution to extend the User data that is inserted on create ?

stunaz•12/15/23, 12:24 AM

what i found is that, it just doesnt work using database as session strategy. its work when you are using jwt as the session strategie, then additionnal fiels are stored with in the jwt token, and you retrieve if from there to add to your session

Josh•12/15/23, 12:34 AM

this works fine in my repo

Josh•12/15/23, 12:35 AM

https://github.com/GentikSolm/t3-app-dir

GitHub

GitHub - GentikSolm/t3-app-dir: t3 app dir boilerplate

t3 app dir boilerplate. Contribute to GentikSolm/t3-app-dir development by creating an account on GitHub.

Josh•12/15/23, 12:35 AM

I'm about to massively overhaul that repo, bit it should show you how to extend it

SSteve Melons Thanks for the feedback, and yes I have done all those things. I figured it out....

Josh•12/15/23, 12:36 AM

it not really a bug as much as it is an adapter issue. they can't do much from their end because they have drizzle orm files inhouse for the user table

Josh•12/15/23, 12:36 AM

so their user data is actually different from yours in the drizzle adapter

Josh•12/15/23, 12:37 AM

this works with extra columns, it just won't grab them unless you explicitly override them which is what I did, and honestly it's not much worth at all and even gives you a better idea of what's going on under the hood

JJosh this works with extra columns, it just won't grab them unless you explicitly ove...

EnricoBaivo•12/15/23, 10:07 AM

Thanks for you responses !
Yeah, due to research i also ended up making my own adapter, but i am struggling with the correct Typeing.

EnricoBaivo•12/15/23, 10:08 AM

Josh•12/15/23, 3:45 PM

just look at the repo I linked and check out how I do it

Dor•12/16/23, 12:59 PM

why is this working out of the box with Prisma but not with drizzle? they need to fix this. I wasted 2 days finding solutions

DDor why is this working out of the box with Prisma but not with drizzle? they need t...

Josh•12/16/23, 5:43 PM

this is not a paid product, it is open source. if it want something fixed that badly do it yourself. these people are working completely for free on these amazing projects. just because it doesn't exactly fit your needs gives you no right to be angry about it

EnricoBaivo•12/20/23, 3:34 PM

I mean i did it also like you did but im using MySQL and i cant return on insert. So i do a select return but this could be null and the create callback dosent allow that:

EEnricoBaivo I mean i did it also like you did but im using MySQL and i cant return on insert...

Josh•12/20/23, 6:37 PM

you shouldnt be mixing .then with await
since we just inserted the user, we can be very sure the user exists in the db, so you can safely tell typescript that it will for sure not be null

Dinocan•1/8/24, 2:14 PM

I found the solution for my project, I hope it will be a solution for you too.

Yiannis•1/8/24, 4:44 PM

I found a solution to this a while ago by julius:
https://github.com/nextauthjs/next-auth/pull/8561

GitHub

feat(adapter-drizzle): add option to pass in schema by juliusmarmin...

Reasoning

In other adapters, the adapter returns the entire table so if you have additional fields they will also be returned when the adapter queries the table. A good example here is if your ...

Yiannis•1/8/24, 4:45 PM

basically overrides a function of the drizzle adaptor. Still need to augment the interface and add the field to the db, but now the adaptor will actually fetch it

Yiannis•1/8/24, 4:48 PM

You need to also make sure you augment the right module
https://github.com/nextauthjs/next-auth/pull/9462

GitHub

fix(docs): updated typescript module imports by Huijiro · Pull Requ...

Reasoning
As discussed in #8948, #9253, and #9377, some augmentation of types in the documentation is wrong, this pull request aims to solve those issues by updating the documentation.
I also ad...

Liltripple_reid•1/8/24, 5:18 PM

hey @Steve Melons (or anyone on this thread) did you by any chance also came across the typeError gave by the drizzle-adapter? I just created a new t3-app and I'm getting this

Steve MelonsOP•1/8/24, 5:29 PM

Does it build when you @ts-ignore ?

Steve MelonsOP•1/8/24, 5:29 PM

I think I had the same issue

Liltripple_reid•1/8/24, 5:29 PM

it does but the sign-in/up flows on my prod website and local are completely broken

Liltripple_reid•1/8/24, 5:29 PM

Liltripple_reid•1/8/24, 5:30 PM

I'm getting this error with credentials I've used before

Liltripple_reid•1/8/24, 5:31 PM

I literally just updated dependencies and started getting a lot of errors from nextauth and the drizzle adapter

Yiannis•1/8/24, 5:33 PM

import type { Adapter } from "next-auth/adapters";

Yiannis•1/8/24, 5:33 PM

Needs a different import I think

Liltripple_reid•1/8/24, 5:46 PM

I have no type Adapter imported here

Liltripple_reid•1/8/24, 5:46 PM

my whole file

Yiannis•1/8/24, 6:02 PM

You may need to augment the type and include the user id

Yiannis•1/8/24, 6:22 PM

@Liltripple_reid
According to this you should just cast it for now. https://github.com/nextauthjs/next-auth/issues/9493

GitHub

Types of property 'createUser' are incompatible · Issue #9493 · nex...

Environment System: OS: Windows 11 10.0.22000 CPU: (32) x64 AMD Ryzen 9 3950X 16-Core Processor Memory: 88.88 GB / 127.94 GB Binaries: Node: 20.6.0 - C:\Program Files\nodejs\node.EXE Yarn: 1.22.19 ...

Liltripple_reid•1/8/24, 7:01 PM

I’ll try that will update soon

Liltripple_reid•1/8/24, 7:35 PM

Update: the ts errors are out, but the sign in errors are still there

Liltripple_reid•1/8/24, 7:35 PM

Yiannis•1/8/24, 7:52 PM

Double check your env variables?

Liltripple_reid•1/8/24, 7:53 PM

100 times

Liltripple_reid•1/8/24, 7:54 PM

same error "Client not found"

Yiannis•1/8/24, 7:55 PM

is your repo public ?

Liltripple_reid•1/8/24, 7:55 PM

wait

Liltripple_reid•1/8/24, 7:55 PM

do discord's or google's client_secret keys expire?