View Issue Details
|ID||Project||Category||View Status||Date Submitted||Last Update|
|0007157||Taler||exchange||public||2022-01-24 22:19||2022-02-19 21:31|
|Reporter||Florian Dold||Assigned To||Christian Grothoff|
|Priority||normal||Severity||minor||Reproducibility||have not tried|
|Product Version||git (master)|
|Target Version||0.9||Fixed in Version||0.9|
|Summary||0007157: deposits: consider not allowing wire_transfer_deadline of "never"|
|Description||The exchange currently allows deposits with wire transfer deadlines of "never".|
This doesn't really make sense. Can we reject deposits with unreasonable wire transfer deadlines or at least truncate them to the deposit deadline of the denomination of the coin we're depositing?
|Tags||No tags attached.|
Fixed in bc3ec563736d28ba73ae4db364086fc30132b730. Note that a 'coin' does not have a deposit deadline, so we cannot truncate to that.
Also, we can only sanely exclude 'forever', as other deadline may make sense. In fact, even 'forever' makes sense, as the transfer would still be triggered when the refund deadline has expired AND the merchant makes another deposit with a non-forever deadline (and aggregation would combine the two and pick the lesser deadline!).
However, most likely a deposit deadline of 'never' is a bug/mistake, and in practice merchants can achieve the same effect by using 100 years into the future or something similarly silly high. So I think forbidding 'never' in mechant API and exchange API is a good thing.
|2022-01-24 22:19||Florian Dold||New Issue|
|2022-01-24 22:19||Florian Dold||Status||new => assigned|
|2022-01-24 22:19||Florian Dold||Assigned To||=> Christian Grothoff|
|2022-02-19 21:31||Christian Grothoff||Status||assigned => resolved|
|2022-02-19 21:31||Christian Grothoff||Resolution||open => fixed|
|2022-02-19 21:31||Christian Grothoff||Fixed in Version||=> 0.9|
|2022-02-19 21:31||Christian Grothoff||Note Added: 0018702|
|2022-02-19 21:31||Christian Grothoff||Product Version||=> git (master)|
|2022-02-19 21:31||Christian Grothoff||Target Version||=> 0.9|