New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Hook up JWT validation #15

Open

spacedub opened this issue May 26, 2023 · 1 comment

Assignees

Labels

Milestone

Contributor

spacedub commented May 26, 2023

Code is here, but has to be put in properly:

https://github.com/codecomet-io/kohoutek/blob/main/app/src/lib/verify-authentication.server.ts#L74

key should be loaded only once, at startup
key should be put in configuration
verification is async in nature and process should be adapted to accommodate that
should be tested... specifically, not clear if the thing just throws or what else it returns

In itself, this is critical, just not right now :)

Tagging @briznad

spacedub added the P0 label

spacedub added this to the Next milestone

spacedub added the enhancement label

spacedub assigned briznad

Contributor Author

spacedub commented Jun 14, 2023

Updated url: https://github.com/codecomet-io/kohoutek/blob/jwt-verification/app/src/lib/verify-authentication.server.ts

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment