View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0005447 | Taler | auditor | public | 2018-09-28 08:47 | 2019-12-20 18:12 |
| Reporter | Florian Dold | Assigned To | Marcello Stanisci | 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 | feature | Reproducibility | have not tried |
| Status | closed | Resolution | fixed | ||
| Product Version | git (master) | ||||
| Target Version | 0.6 | Fixed in Version | 0.6 | ||
| Summary | 0005447: auditor should accept samples of deposit confirmations from merchants | ||||
| Description | ... to then be able to revoke signing keys, if deposit confirmations were observed that do not match or even contradict the exchange's database. | ||||
| Tags | No tags attached. | ||||
| The HTTP API has been specified and is (largely) implemented (but not tested). Mainly missing are: (0) (more) testing of the API (1) actual use during auditing (2) deployment of an HTTPd for the auditor (3) actual use by the merchants | |
| (1) is now done, patch attached, could not yet push as git.taler.net:22 seems down... | |
| Started with logic to call auditor for dc's in exchange-lib in 2a01aaa9..12123a7a (logic incomplete). | |
| 91566dde..7cc7660c concludes the implementation (step #3). However, more testing would still be good. Marcello: please write a test so that it performs a few deposit operations (10x) with an exchange (httpd) and auditor (httpd), *and* subsequently runs the taler-auditor so that we have a high chance of covering the new logic. | |
| By better looking at the code, commit 537af4631d922b2190b3a3ce8fb.. does implement the test requested in the previous note. In fact, the batch named "massive-deposit-confirms" runs the 10x deposit operations and the one called "normal-auditor" runs taler-auditor. | |
| 7b6488d7d77e5da3e7 adds a taler-auditor test command ("massive-auditor") right inside the block with the 10x deposit operations, so this should be solved now. | |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2018-09-28 08:47 | Florian Dold | New Issue | |
| 2018-09-28 08:47 | Florian Dold | Status | new => assigned |
| 2018-09-28 08:47 | Florian Dold | Assigned To | => Christian Grothoff |
| 2018-10-06 13:08 | Christian Grothoff | Severity | minor => feature |
| 2018-10-28 10:41 | Christian Grothoff | Note Added: 0013292 | |
| 2018-11-04 16:39 | Christian Grothoff | Note Added: 0013319 | |
| 2018-11-17 23:33 | Christian Grothoff | Product Version | => git (master) |
| 2018-11-17 23:33 | Christian Grothoff | Target Version | => 0.7.1 |
| 2019-01-10 15:20 | Christian Grothoff | Note Added: 0013455 | |
| 2019-01-13 16:49 | Christian Grothoff | Note Added: 0013457 | |
| 2019-01-13 16:49 | Christian Grothoff | Assigned To | Christian Grothoff => Marcello Stanisci |
| 2019-01-30 22:01 | Marcello Stanisci | Note Added: 0013527 | |
| 2019-01-30 22:21 | Marcello Stanisci | Note Added: 0013528 | |
| 2019-01-30 22:25 | Marcello Stanisci | Status | assigned => informing |
| 2019-01-30 22:25 | Marcello Stanisci | Resolution | open => fixed |
| 2019-01-31 18:33 | Christian Grothoff | Fixed in Version | => 0.6 |
| 2019-01-31 18:33 | Christian Grothoff | Target Version | 0.7.1 => 0.6 |
| 2019-12-20 18:12 | Christian Grothoff | Status | informing => closed |