View Issue Details

IDProjectCategoryView StatusLast Update
0007793Talerotherpublic2023-09-23 13:09
ReporterFlorian Dold Assigned Tosebasjm Resolution deadline
ImplementerReviewer 
Release signed byDeployed by 
VerifierCommunicator 
Data protection authority notification requiredNo  
CSIRT notification requiredNo  
Estimated effort (hours)Actual effort (hours)0 
Fast-trackNo
Work packagenone
PrioritynormalSeverityfeatureReproducibilityhave not tried
Status closedResolutionfixed 
Product Versiongit (master) 
Target Version0.9.3Fixed in Version0.9.3 
Summary0007793: use token auth for SPAs instead of storing credentials plain-text in local storage
DescriptionWhat we currently do is obviously not very nice from a security perspective.

Instead of using the plain-text user/password credentials, there should be a /login endpoint that then returns a bearer token.
TagsNo tags attached.

Required notifications

Address or linkRemarkAdded byNotifiedActions

Activities

Christian Grothoff

2023-09-05 12:04

manager   ~0020500

Last edited: 2023-09-05 12:11

This will require a token table with the list of currently valid access tokens per account/instance. -- or maybe we just sign the token and store all of the data inside the token!

Christian Grothoff

2023-09-05 12:15

manager   ~0020501

access token should include account/instance, scope, creation timestamp (for forced logout per account) and expiration + EdDSA signature.

Christian Grothoff

2023-09-05 16:05

manager   ~0020505

3f86f293..47a86dfc specifies the /login API for this. Not yet implemented.

Christian Grothoff

2023-09-06 20:42

manager   ~0020508

Now implemented in C backend and tests pass. SPA missing.

Issue History

Date ModifiedUsernameFieldChange
2023-04-05 16:40Florian DoldNew Issue
2023-09-05 12:02Christian GrothoffAssigned To => Christian Grothoff
2023-09-05 12:02Christian GrothoffStatusnew => assigned
2023-09-05 12:02Christian GrothoffSeverityminor => feature
2023-09-05 12:02Christian GrothoffProduct Version => git (master)
2023-09-05 12:02Christian GrothoffTarget Version1.0 => 0.9.3
2023-09-05 12:04Christian GrothoffNote Added: 0020500
2023-09-05 12:11Christian GrothoffNote Edited: 0020500
2023-09-05 12:15Christian GrothoffNote Added: 0020501
2023-09-05 16:05Christian GrothoffNote Added: 0020505
2023-09-06 20:42Christian GrothoffNote Added: 0020508
2023-09-06 20:42Christian GrothoffAssigned ToChristian Grothoff => sebasjm
2023-09-11 19:19sebasjmStatusassigned => informing
2023-09-11 19:19sebasjmResolutionopen => fixed
2023-09-23 13:07Christian GrothoffFixed in Version => 0.9.3
2023-09-23 13:09Christian GrothoffStatusinforming => closed