search

DarrenBaldwin07 / clerk-rs

An unofficial Rust SDK for Clerk.dev
https://crates.io/crates/clerk-rs
MIT License
58 stars 23 forks source link

[feature] make sid optional #48

Closed after-ephemera closed 2 weeks ago

after-ephemera commented 1 month ago

In doing some testing with long lived tokens I realized that clerk-rs expects the sid field to be present when decoding JWTs. However, long-lived template tokens don't appear to have that field on the payload. From jwt.io decoding of a long-lived token of mine:

Screenshot 2024-06-03 at 8 04 27 AM

I propose making the sid field optional.

after-ephemera commented 1 month ago

Using JS in the browser according to the article. I don't yet have a client, so I'm just requesting a template token.

DarrenBaldwin07 commented 4 weeks ago

Using JS in the browser according to the article. I don't yet have a client, so I'm just requesting a template token.

https://github.com/DarrenBaldwin07/clerk-rs/pull/49#pullrequestreview-2101879374