View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0009098 | Taler | exchange | public | 2024-08-19 17:16 | 2024-09-17 14:29 |
| Reporter | Florian Dold | Assigned To | Florian Dold | Resolution deadline | |
| Implementer | Reviewer | ||||
| Release signed by | Deployed by | ||||
| Verifier | Communicator | ||||
| Data protection authority notification required | No | ||||
| CSIRT notification required | No | ||||
| Estimated effort (hours) | Actual effort (hours) | 0 | |||
| Fast-track | No | ||||
| Work package | none | ||||
| Priority | normal | Severity | block | Reproducibility | N/A |
| Status | closed | Resolution | fixed | ||
| Product Version | git (master) | ||||
| Target Version | 0.14 | Fixed in Version | 0.14 | ||
| Summary | 0009098: deposit works despite OPERATION_TYPE=AGGREGATE rule that should trigger AML | ||||
| Description | The exchange's `deposits/...` endpoint for a deposit of TESTKUDOS:10 in total (multiple coins!) returns 200 (wired) for all coins with kyc_ok=true. However, according to the KYC config, there is a TESTKUDOS:5 deposit threshold. | ||||
| Steps To Reproduce | $ taler-harness run-integrationtests kyc-deposit-aggregate (Note that the test is *not* expected to succeed yet even with a correct exchange, as the deposit implementation does not fully support the new KYC API yet) | ||||
| Additional Information | [exchange] enable_kyc = yes [KYC-RULE-R1] OPERATION_TYPE = aggregate ENABLED = yes EXPOSED = yes IS_AND_COMBINATOR = yes THRESHOLD = TESTKUDOS:5 TIMEFRAME = 1d NEXT_MEASURES = M1 [KYC-MEASURE-M1] CHECK_NAME = C1 CONTEXT = {} PROGRAM = P1 [AML-PROGRAM-P1] COMMAND = /bin/true ENABLED = true DESCRIPTION = this does nothing FALLBACK = M1 [KYC-CHECK-C1] TYPE = INFO DESCRIPTION = my check! FALLBACK = M1 | ||||
| Tags | No tags attached. | ||||
| Could you please make sure to log taler-exchange-aggregator at the INFO level during the test? I also added more logging logic to help diagnose this. | |
| Test works in taler-harness passes now. | |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2024-08-19 17:16 | Florian Dold | New Issue | |
| 2024-08-19 17:16 | Florian Dold | Status | new => assigned |
| 2024-08-19 17:16 | Florian Dold | Assigned To | => Christian Grothoff |
| 2024-08-22 20:08 | Christian Grothoff | Note Added: 0023034 | |
| 2024-08-22 20:09 | Christian Grothoff | Assigned To | Christian Grothoff => Florian Dold |
| 2024-08-22 20:09 | Christian Grothoff | Status | assigned => feedback |
| 2024-08-28 21:28 | Florian Dold | Target Version | 0.13 => 0.14 |
| 2024-09-05 17:13 | Florian Dold | Status | feedback => informing |
| 2024-09-05 17:13 | Florian Dold | Resolution | open => fixed |
| 2024-09-05 17:13 | Florian Dold | Note Added: 0023181 | |
| 2024-09-05 17:27 | Christian Grothoff | Product Version | => git (master) |
| 2024-09-05 17:27 | Christian Grothoff | Fixed in Version | => 0.14 |
| 2024-09-17 14:29 | Christian Grothoff | Status | informing => closed |