View Issue Details

IDProjectCategoryView StatusLast Update
0008916Talerlibeufin-bankpublic2025-05-08 20:41
ReporterMarcS Assigned ToResolution 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
PrioritylowSeverityfeatureReproducibilityhave not tried
Status implementingResolutionopen 
Product Version0.12 
Target Versionpost-1.0 
Summary0008916: Idle logout of demo bank website
DescriptionI experienced that when trying to open bank.head.taler.net, the website showed an "internal error". Probably because my browser still sent an old session cookie but HEAD got updated. After I logged out, I could login again and it worked.

==> instead of showing some internal error, the website should directly go to the login dialog if the session is unknown (or timed out). No need to bug the user with more steps (clicking away that nasty error message plus manually logout) since they need to login anyway.

ALL commercial banks do logout users after 15min idle. We should do the same, and dismiss the session after 15min without user input. Please show a count-down telling the user when the session will be terminated, and reset that back to 15 minutes on each user action.
TagsNo tags attached.

Required notifications

Address or linkRemarkAdded byNotifiedActions

Relationships

related to 0009572 closedsebasjm automatic logout (est 4h - due date 5/6) 
related to 0008942 implementing check /config when doing a request to account-info-endpoint returns 400 [4hs] 

Activities

sebasjm

2024-06-13 16:28

developer   ~0022597

after meeting we decided to split this into two issues:

 * handling http resp 400 with logout in some cases, it will be tracked with issue 0008916
 * enforcing short session like a commercial bank, will be tracked with this issue, implemented server side.

Antoine A

2024-06-16 14:58

manager   ~0022621

Maybe we should use a specific Taler error code for those cases

Christian Grothoff

2024-07-25 22:06

manager   ~0022850

Sure, never wrong to add more specific error codes.

sebasjm

2024-08-05 13:32

developer   ~0022921

If we implement it client-side (SPA), it can be bypassed by a custom client and doesn't really fix anything. From the user PoV it's better for the session to be kept as long as possible so it doesn't need to input the password again and again (unless this bring security problem).

If we implement it server-side, this would mean restricting the duration of all tokens created by common users (even the "forever" ones). Since we still need real "forever" tokens for exchange, merchant, anastasis, etc., so I suggests to only apply this to non-exchange users.

Marked as post-1.0 nice to have

Christian Grothoff

2025-05-08 20:39

manager   ~0024853

Sebastian: logout of idle sessions is sometimes a *legal* requirement. And the fact that users can bypass this via a custom SPA is hardly an argument, as it presumably protects their sessions from being stolen.

Christian Grothoff

2025-05-08 20:41

manager   ~0024854

I suggest we implement this at the same time when we fix 0009572 as the same logic should apply.

Issue History

Date ModifiedUsernameFieldChange
2024-06-09 14:53MarcSNew Issue
2024-06-09 14:53MarcSStatusnew => assigned
2024-06-09 14:53MarcSAssigned To => sebasjm
2024-06-09 14:56MarcSDescription Updated
2024-06-12 16:37sebasjmPrioritynormal => low
2024-06-12 16:37sebasjmSeveritymajor => minor
2024-06-12 16:37sebasjmTarget Version0.12 => 0.13
2024-06-13 16:16sebasjmAssigned Tosebasjm => Antoine A
2024-06-13 16:16sebasjmCategoryWeb site(s) => libeufin-bank
2024-06-13 16:18sebasjmRelationship addedrelated to 0008942
2024-06-13 16:28sebasjmNote Added: 0022597
2024-06-16 14:58Antoine AStatusassigned => feedback
2024-06-16 14:58Antoine ANote Added: 0022621
2024-07-25 22:06Christian GrothoffNote Added: 0022850
2024-07-28 19:49Christian GrothoffSeverityminor => feature
2024-08-05 13:30sebasjmTarget Version0.13 => post-1.0
2024-08-05 13:32sebasjmNote Added: 0022921
2025-05-08 20:39Christian GrothoffNote Added: 0024853
2025-05-08 20:40Christian GrothoffRelationship addedrelated to 0009572
2025-05-08 20:41Christian GrothoffNote Added: 0024854
2025-05-08 20:41Christian GrothoffAssigned ToAntoine A =>
2025-05-08 20:41Christian GrothoffStatusfeedback => implementing