View Issue Details

IDProjectCategoryView StatusLast Update
0006192Talerexchangepublic2021-09-02 16:14
ReporterMarcello Stanisci Assigned ToChristian Grothoff Resolution deadline
ImplementerReviewer 
Release signed byDeployed by 
VerifierCommunicator 
Data protection authority notification requiredNo  
CSIRT notification requiredNo  
Estimated effort (hours)Actual effort (hours)0 
Fast-trackNo
Work packagenone
PrioritynormalSeverityminorReproducibilityhave not tried
Status closedResolutionfixed 
Product Versiongit (master) 
Target Version0.7.1Fixed in Version0.7.1 
Summary0006192: Avoid substituting valid HTTP status code with 0, when a condition fails.
DescriptionThe http client artifact used by the exchange _substitutes_ a valid response status code with 0 whenever the JSON parser is not
happy about something. This way is harder to understand what happened. For example, a server might have returned 404 but didn't include a 'ec' field in the response, and the testcase ends everything saying that the response had '0' as status code.
Additional InformationThe problem was observed with a invalid JSON, but that doesn't mean it doesn't exist when other conditions could fail.
TagsNo tags attached.

Required notifications

Address or linkRemarkAdded byNotifiedActions

Activities

Christian Grothoff

2020-04-21 11:58

manager   ~0015722

Fixed in a5460cc8..c167fd8f

Christian Grothoff

2021-09-02 16:14

manager   ~0018261

Fix committed to master branch.

Issue History

Date ModifiedUsernameFieldChange
2020-04-21 11:39Marcello StanisciNew Issue
2020-04-21 11:39Marcello StanisciStatusnew => assigned
2020-04-21 11:39Marcello StanisciAssigned To => Christian Grothoff
2020-04-21 11:58Christian GrothoffStatusassigned => informing
2020-04-21 11:58Christian GrothoffResolutionopen => fixed
2020-04-21 11:58Christian GrothoffFixed in Version => 0.7.1
2020-04-21 11:58Christian GrothoffNote Added: 0015722
2020-04-21 11:58Christian GrothoffProduct Version => git (master)
2020-04-21 11:58Christian GrothoffTarget Version => 0.7.1
2021-08-24 14:23Christian GrothoffStatusinforming => closed
2021-09-02 16:13Christian Grothoff
2021-09-02 16:14Christian GrothoffNote Added: 0018261