View Issue Details

IDProjectCategoryView StatusLast Update
0004803Talerexchangepublic2016-11-20 02:26
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.2Fixed in Version0.2 
Summary0004803: /admin/add/incoming: non representative error message
DescriptionIf the /admin/add/incoming handler does not accept the bank specified in
the 'sender_account_details' field of the input JSON, then it returns the
following error to the caller:

"{\n \"error\": \"unknown entity referenced\",\n \"code\": 1600,\n \"parameter\": \"sender_account_details\"\n}"

In contrast, for the same error, it logs the more representative message:

Nov 18 15:00:39-697838 taler-exchange-httpd-8347 ERROR Wire specifies bank URI http://localbank/, but this exchange only supports https://bank.demo.taler.net/
Nov 18 15:00:39-697867 taler-exchange-httpd-8347 WARNING External protocol violation detected at taler-exchange-httpd_admin.c:89.

The error returned to the caller sounds more like a failure in the parsing, rather than the
exchange just doesn't accept the bank in 'sender_account_details'.
Steps To ReproduceTry to with withdraw from a bank not accepted by the exchange.
TagsNo tags attached.

Required notifications

Address or linkRemarkAdded byNotifiedActions

Activities

Christian Grothoff

2016-11-18 17:29

manager   ~0011531

Fixed in de68a7b..7d6b8d5

Issue History

Date ModifiedUsernameFieldChange
2016-11-18 14:21Marcello StanisciNew Issue
2016-11-18 14:21Marcello StanisciStatusnew => assigned
2016-11-18 14:21Marcello StanisciAssigned To => Christian Grothoff
2016-11-18 15:07Christian GrothoffProduct Version => git (master)
2016-11-18 15:07Christian GrothoffFixed in Version => 0.2
2016-11-18 15:07Christian GrothoffTarget Version => 0.2
2016-11-18 17:29Christian GrothoffStatusassigned => informing
2016-11-18 17:29Christian GrothoffResolutionopen => fixed
2016-11-18 17:29Christian GrothoffNote Added: 0011531
2016-11-20 02:26Christian GrothoffStatusinforming => closed