View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0006921 | Taler | merchant backend | public | 2021-07-13 04:46 | 2024-01-12 13:04 |
| Reporter | Christian Grothoff | 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 | major | Reproducibility | always |
| Status | closed | Resolution | fixed | ||
| Platform | i7 | OS | Debian GNU/Linux | OS Version | squeeze |
| Product Version | git (master) | ||||
| Target Version | 0.8 | Fixed in Version | 0.8 | ||
| Summary | 0006921: merchant backend does not accept payments for 0-valued contracts | ||||
| Description | Florian writes: I've just added an integration test for this, fixed processing of zero-value orders in the wallet and improved error reporting during DB transactions a bit. The merchant, however, doesn't like /pay with zero coins *at all*: It doesn't even return a proper error message, but just resets the connection! Jul 12 15:52:55-824536 taler-merchant-httpd-63937(58CZPKRCHKDFWNHEVC6ZEBNTCM) ERROR Assertion failed at taler-merchant-httpd_post-orders-ID-pay.c:1071. Aborting. I would argue that the merchant *ought* to accept this request, for two reasons: 1. It allows the wallet to obtain the payment signature. Sure, the product might've been free (e.g. 100% special discount), but the wallet must still be able to prove that it paid for it, *not* just claimed it 2. Related, the session-bound payments also need /pay to work, think of a "temporarily 100% discounted blog article". You can easily reproduce the merchant issue with the latest git wallet by running $ taler-wallet-cli testing run-integrationtests payment-zero - Florian | ||||
| Tags | No tags attached. | ||||
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2021-07-13 04:46 | Christian Grothoff | New Issue | |
| 2021-07-13 04:46 | Christian Grothoff | Status | new => assigned |
| 2021-07-13 04:46 | Christian Grothoff | Assigned To | => Christian Grothoff |
| 2021-07-13 04:46 | Christian Grothoff | Assigned To | Christian Grothoff => |
| 2021-07-13 04:46 | Christian Grothoff | Status | assigned => implementing |
| 2021-07-22 17:04 | Florian Dold | Assigned To | => Florian Dold |
| 2021-07-22 17:04 | Florian Dold | Status | implementing => informing |
| 2021-07-22 17:04 | Florian Dold | Resolution | open => fixed |
| 2021-07-22 17:04 | Florian Dold | Note Added: 0018005 | |
| 2021-07-30 11:56 | Christian Grothoff | Fixed in Version | => 0.8 |
| 2021-07-30 11:59 | Christian Grothoff | Target Version | => 0.8.1 |
| 2021-07-30 12:01 | Christian Grothoff | Target Version | 0.8.1 => 0.8 |
| 2021-08-24 14:22 | Christian Grothoff | Status | informing => closed |
| 2024-01-12 13:04 | Christian Grothoff | Category | merchant backend API (C) => mechant backend |
| 2025-03-22 13:09 | Christian Grothoff | Category | mechant backend => merchant backend |