View Issue Details

IDProjectCategoryView StatusLast Update
0004771Talerwallet (WebExtension)public2018-04-15 18:33
ReporterMarcello Stanisci Assigned ToFlorian Dold 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
PrioritynormalSeveritymajorReproducibilityhave not tried
Status closedResolutionfixed 
Product Versiongit (master) 
Target Version0.5Fixed in Version0.5 
Summary0004771: Error code swallowed.
DescriptionThe merchant frontend says nothing about the following error. Indagate where the error code
got swallowed: could be backend/frontend/or even exchange.

Nov 09 16:02:57-006923 taler-exchange-httpd-5943 ERROR Wire specifies bank URI https://bank.test.taler.net/, but this exchange only supports http://localbank/
Nov 09 16:02:57-007139 taler-merchant-httpd-5946 WARNING Deposit operation failed with HTTP code 404
Steps To ReproducePick a Taler installation, change fsf.json so that it contains a "bank_uri" field that differs from exchange's and try to buy a blog article.

Then see the error logged in the console.
TagsNo tags attached.

Required notifications

Address or linkRemarkAdded byNotifiedActions

Activities

Marcello Stanisci

2017-07-28 08:10

reporter   ~0012369

The frontend just responds to /pay by echoing what the exchange says - just reproduced. Since the communication is Wallet<->Frontend, it is wallet responsibility to inform the user via its UI.

Note that the error *is* handled, but it's just logged in the console.. I'm going to change category and assigned person.

Christian Grothoff

2018-01-30 07:30

manager   ~0012837

Should this be fixed for 0.5?

Florian Dold

2018-01-30 11:34

manager   ~0012840

This was already "silently" fixed, the merchant frontends now always have nice, informative error pages when something with a backend request goes wrong.

Issue History

Date ModifiedUsernameFieldChange
2016-11-09 16:59Marcello StanisciNew Issue
2016-11-09 16:59Marcello StanisciStatusnew => assigned
2016-11-09 16:59Marcello StanisciAssigned To => Marcello Stanisci
2016-11-15 14:39rootProduct Version => git (master)
2016-11-15 14:39rootTarget Version => 0.3
2017-03-05 23:26Christian GrothoffSeverityminor => major
2017-05-02 12:44Christian GrothoffTarget Version0.3 => 0.7.1
2017-07-28 08:10Marcello StanisciNote Added: 0012369
2017-07-28 08:12Marcello StanisciAssigned ToMarcello Stanisci => Florian Dold
2017-07-28 08:12Marcello StanisciCategorymechant backend => wallet (WebExtensions)
2017-07-28 08:12Marcello StanisciSteps to Reproduce Updated
2018-01-30 07:30Christian GrothoffNote Added: 0012837
2018-01-30 11:34Florian DoldStatusassigned => informing
2018-01-30 11:34Florian DoldResolutionopen => fixed
2018-01-30 11:34Florian DoldNote Added: 0012840
2018-01-30 13:19Florian DoldTarget Version0.7.1 => 0.5
2018-01-31 10:59Christian GrothoffFixed in Version => 0.5
2018-04-15 18:33Christian GrothoffStatusinforming => closed
2023-04-13 18:37Florian DoldCategorywallet (WebExtensions) => wallet (WebExtension)