View Issue Details

IDProjectCategoryView StatusLast Update
0009885Talerexchangepublic2025-11-27 16:18
ReporterFlorian Dold 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
PrioritylowSeveritytweakReproducibilityhave not tried
Status implementingResolutionopen 
Target Versionpost-1.0 
Summary0009885: AML programs should validate the FORM_ID and form attributes
DescriptionThe AML programs we use in the exchange currently do not check the FORM_ID at all.

For programs that run after a form submission, we do some more basic checks on the new attributes.
Tagsdebt

Required notifications

Address or linkRemarkAdded byNotifiedActions

Relationships

has duplicate 0010552 closed VQF 902.1: AML program should check required forms were fully submitted 

Activities

Christian Grothoff

2025-06-22 10:50

manager   ~0025300

dev/grothoff/bug9885 contains a patch that *could* in *theory* fix this. It is, however, largely untested. It also has some specific known issues:

(1) I could not find the name of the "accept ToS" form. So I put "accept_tos" into taler-exchange-helper-measure-validate-accepted-tos. Sebastian: please double-check that the ToS dialog actually creates a form ID (the logic seems to be missing), and if it does have a different one already, please adjust the shell script.

(2) The kyx-check script doesn't yet check the "innards" of the ESTABLISHER_LIST.

(3) The 'reject' logic (exec taler-exchange-helper-measure-freeze) should probably be tested...

sebasjm

2025-06-23 02:46

developer   ~0025314

form id fixed in the script, i think we should have the form_id in gana 0010122

Christian Grothoff

2025-07-03 15:25

manager   ~0025404

Merged branch for further testing in packages.

Issue History

Date ModifiedUsernameFieldChange
2025-05-06 01:51Florian DoldNew Issue
2025-05-08 17:26Christian GrothoffStatusnew => implementing
2025-05-08 17:27Christian GrothoffSeverityminor => tweak
2025-05-08 19:57Christian GrothoffTarget Version1.0 stretch goals => post-1.0
2025-05-08 19:57Christian GrothoffPrioritynormal => low
2025-06-08 18:21Christian GrothoffTag Attached: debt
2025-06-16 00:05Christian GrothoffAssigned To => Christian Grothoff
2025-06-16 00:05Christian GrothoffStatusimplementing => assigned
2025-06-22 10:50Christian GrothoffNote Added: 0025300
2025-06-23 02:46sebasjmNote Added: 0025314
2025-07-03 15:25Christian GrothoffNote Added: 0025404
2025-07-04 09:55Christian GrothoffAssigned ToChristian Grothoff => Florian Dold
2025-07-29 22:27Christian GrothoffAssigned ToFlorian Dold =>
2025-07-29 22:27Christian GrothoffStatusassigned => implementing
2025-07-29 22:27Christian GrothoffTarget Versionpost-1.0 => 1.4
2025-11-27 16:14Christian GrothoffTarget Version1.4 => post-1.0
2025-11-27 16:18Christian GrothoffRelationship addedhas duplicate 0010552