Skip to content

Conversation

@izadoesdev
Copy link

No description provided.

@vercel
Copy link

vercel bot commented Sep 8, 2025

@izadoesdev is attempting to deploy a commit to the Vercel Team on Vercel.

A member of the Team first needs to authorize it.

@sergioxro
Copy link

damn

@izadoesdev
Copy link
Author

all functionality intact, everything works, just needs database migration

@techwithanirudh
Copy link

techwithanirudh commented Sep 9, 2025

const session = await auth.api.getSession({ headers: await headers() });

instead of calling auth.api.getSession every time, i think it'd be cleaner if you define a getSession function?

export const getSession = async (request?: NextRequest) => {
  return auth.api.getSession({
    headers: request ? request.headers : await headers(),
  })
}

other than that, looks fine

@socket-security
Copy link

socket-security bot commented Sep 9, 2025

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Added@​types/​pg@​8.15.51001007186100
Addedpg@​8.16.3991009984100
Addedbetter-auth@​1.3.8991008596100
Updateddrizzle-orm@​0.34.1 ⏵ 0.44.59810087 +199 +1100
Updateddrizzle-kit@​0.25.0 ⏵ 0.31.499 +110096 +199 +1100
Updated@​biomejs/​biome@​1.9.4 ⏵ 2.2.3100 +710010099 +2100

View full report

@okanji
Copy link

okanji commented Oct 8, 2025

Would love to see this go through

@dancer
Copy link
Contributor

dancer commented Nov 1, 2025

we will migrate to better-auth soon but this pr has to many merge conflict so will do it ourself but thanks iza

@dancer dancer closed this Nov 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants