Skip to content

feat: EdDSA token for database leakage/index mitigation#971

Draft
eternal-flame-AD wants to merge 1 commit into
masterfrom
pk-token
Draft

feat: EdDSA token for database leakage/index mitigation#971
eternal-flame-AD wants to merge 1 commit into
masterfrom
pk-token

Conversation

@eternal-flame-AD
Copy link
Copy Markdown
Member

Fixes #325 , supercedes #707

Backend should be working with tests now, I'm currently settling on a timestamp based signature scheme like TOTP with more relaxed window (+-15 minutes for now). Some early feedback would be appreciated

TODO:

  • make the browser frontend sign tokens. It will probably work something like this: when you login (or every time you reopen the web page) it will attempt to sign the request, if the server rejects that we try a second time without signing it
  • UI improvement: amend the schema to differentiate errors between invalid tokens and expired signature

also refactored some tests that used to rely on hijacking the token generation function to be consistency/property based.

@jmattheis
Copy link
Copy Markdown
Member

Could you answer the questions in #966 (comment)?

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

Security: Store api token hashed

2 participants