View Issue Details

IDProjectCategoryView StatusLast Update
0008499Talerlibeufin-bankpublic2024-03-07 19:49
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
PrioritynormalSeverityminorReproducibilityhave not tried
Status closedResolutionfixed 
Target Version0.9.4Fixed in Version0.9.4 
Summary0008499: bank UI allows admin to cash-out for users, but backend forbids it
DescriptionNot sure which component is wrong, the UI or the backend. Should the admin be able to start cash-outs on behalf of other accounts?
TagsNo tags attached.

Required notifications

Address or linkRemarkAdded byNotifiedActions

Activities

Antoine A

2024-02-24 17:17

manager   ~0021486

The backend doesn't allow the administrator to perform bank transactions or cashouts for another user, but I can easily change that.

Christian Grothoff

2024-02-25 17:46

manager   ~0021492

I'm not sure we should! I think the main thing is that the GUI needs to allow normal users to cash-out.

Florian Dold

2024-02-25 18:07

manager   ~0021500

I think it does allow cash-outs for normal users, it was just so well hidden in the UI that I did not find them ;-)

Christian Grothoff

2024-02-25 18:24

manager   ~0021502

Last edited: 2024-02-25 18:24

Is this now a duplicate of 0008498? Well, I guess NOT allowing the admin (in the UI) to do cash-out for other users is then this bug, right? (In the backend, my understanding is that it is already forbidden.)

sebasjm

2024-02-27 12:19

developer   ~0021533

i'm not sure if I understood this correctly.

I'm removing the link "cashouts" in the account list in the admin home page. This remove the access for the admin to the user cashout. Previously, this was a place to look for the list of cashouts of the user (and it worked) now I see that backend forbids it. (if my memory works ok)

sebasjm

2024-02-27 12:19

developer   ~0021534

please check that the change is what you expected

Florian Dold

2024-03-01 11:11

manager   ~0021599

LGTM now.

Issue History

Date ModifiedUsernameFieldChange
2024-02-22 21:53Florian DoldNew Issue
2024-02-22 21:53Florian DoldStatusnew => assigned
2024-02-22 21:53Florian DoldAssigned To => Antoine A
2024-02-24 17:17Antoine ANote Added: 0021486
2024-02-25 17:46Christian GrothoffNote Added: 0021492
2024-02-25 17:58Antoine AAssigned ToAntoine A => sebasjm
2024-02-25 18:07Florian DoldNote Added: 0021500
2024-02-25 18:24Christian GrothoffNote Added: 0021502
2024-02-25 18:24Christian GrothoffNote Edited: 0021502
2024-02-27 12:19sebasjmNote Added: 0021533
2024-02-27 12:19sebasjmAssigned Tosebasjm => Florian Dold
2024-02-27 12:19sebasjmStatusassigned => feedback
2024-02-27 12:19sebasjmNote Added: 0021534
2024-03-01 11:11Florian DoldAssigned ToFlorian Dold => sebasjm
2024-03-01 11:11Florian DoldStatusfeedback => informing
2024-03-01 11:11Florian DoldResolutionopen => fixed
2024-03-01 11:11Florian DoldFixed in Version => 0.9.4
2024-03-01 11:11Florian DoldNote Added: 0021599
2024-03-07 19:49Christian GrothoffStatusinforming => closed