redwoodjs / redwood

The App Framework for Startups
https://redwoodjs.com
MIT License
17.32k stars 994 forks source link

[Auth] Clerk GetToken returning wrong value #4793

Closed BurnedChris closed 2 years ago

BurnedChris commented 2 years ago

Just had a debugging conversation with Sokratis Vidros at Clerk

Get token should return the JWT token but instead it the session id

Wrong: https://github.com/redwoodjs/redwood/blob/3943bd1a1653fb76adca0306c115dcfd22954998/packages/auth/src/authClients/clerk.ts#L39

Correct: It Needs to be clerk.session?.getToken()

thedavidprice commented 2 years ago

@devchampian Is this something you could take a look at?

dthyresson commented 2 years ago

I chatted with Clerk and they said that this implementation isn’t 100% correct and instead they’ll be upgrading the client in Redwood next week to the latest version that will not use the session.

thedavidprice commented 2 years ago

next week

😬

Tobbe commented 2 years ago

Work started over here https://github.com/redwoodjs/redwood/pull/4846