search

lucia-auth / lucia

Authentication, simple and clean
https://lucia-auth.com
MIT License
8.96k stars 460 forks source link

Fix drizzle adapter setSession to work with cloudflare workers #1384

Closed devakrishna33 closed 6 months ago

devakrishna33 commented 7 months ago

If we do not call execute explicitly in drizzle adapter it causes issues when deploying to cloudflare workers. Refer to: https://github.com/cloudflare/next-on-pages/issues/499

pilcrowOnPaper commented 7 months ago

Is this only an issue for insert() or do we need to fix select() etc as well?

devakrishna33 commented 7 months ago

This seems to affect insert, update and delete, I have changed it in my project it works fine, I'm not sure if this will cause any other issues.

pilcrowOnPaper commented 7 months ago

Can you update the Postgres and MySQL ones as well?

saturnonearth commented 6 months ago

Looks like there might be a fix for this on Cloudflare workers side? If I understand it correctly

https://github.com/cloudflare/workerd/issues/1513#issuecomment-1936275341

philippviereck commented 6 months ago

this is now fixed on workers: https://github.com/cloudflare/workerd/issues/1513#issuecomment-1962238498

pilcrowOnPaper commented 6 months ago

Closing this since it seems to be fixed on Cloudflare's end