View Issue Details

IDProjectCategoryView StatusLast Update
0006478Talerexchange API (C)public2020-08-13 00:51
ReporterChristian Grothoff Assigned ToChristian Grothoff  
PriorityhighSeverityminorReproducibilityN/A
Status resolvedResolutionfixed 
Platformi7OSDebian GNU/LinuxOS Versionsqueeze
Product Versiongit (master) 
Target Version0.8Fixed in Version0.8 
Summary0006478: refund API fails to verify 409 and 412 error responses from exchange
DescriptionThe exchange provides proof of the conflict/failure, but the client currently just believes it without checking.
TagsNo tags attached.

Activities

Christian Grothoff

2020-08-12 20:19

manager   ~0016594

26f72f85..6256bdb1 implements the checks, but the logic is untested AND a possible bug found while implementing shows that another major code path is not tested as well. I put an abort() + FIXME to mark the so far untested code. So we need better tests here.

Christian Grothoff

2020-08-13 00:51

manager   ~0016595

6256bdb1..4e0b5104 adds the missing tests and fixes the discovered issues on the now covered paths.

Issue History

Date Modified Username Field Change
2020-08-12 12:13 Christian Grothoff New Issue
2020-08-12 12:13 Christian Grothoff Status new => assigned
2020-08-12 12:13 Christian Grothoff Assigned To => Christian Grothoff
2020-08-12 20:19 Christian Grothoff Note Added: 0016594
2020-08-13 00:51 Christian Grothoff Note Added: 0016595
2020-08-13 00:51 Christian Grothoff Status assigned => resolved
2020-08-13 00:51 Christian Grothoff Resolution open => fixed
2020-08-13 00:51 Christian Grothoff Fixed in Version => 0.8
2020-08-13 00:51 Christian Grothoff Target Version 0.8.1 => 0.8