View Issue Details

IDProjectCategoryView StatusLast Update
0004348Talerwallet (WebExtension)public2016-11-08 12:29
ReporterTorsten Grothoff Assigned ToMarcello Stanisci 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
PriorityimmediateSeverityblockReproducibilityhave not tried
Status closedResolutionduplicate 
Product Version0.0 
Target Version0.0 
Summary0004348: fail to wthtraw pudos
Descriptionween i withdraw pudos the following appers:
Shop Bank About
Error

Exchange unknown

Given exchange does not exist (try 'http' instead of 'https'?)
TagsNo tags attached.

Required notifications

Address or linkRemarkAdded byNotifiedActions

Relationships

related to 0004354 closedFlorian Dold exchange bank account number when withdrawing 

Activities

Marcello Stanisci

2016-03-31 14:55

reporter   ~0010344

the problem is that there is no "https" exchange bank account, so the
script can't find one when trying to wire funds to its (non-existent)
bank account.

This leads to another problem since the "https" exchange URL has 31 chars,
and this lenght is not liked by the 'User' model. That calls for a workaround
until the wallet itself will communicate exchange's bank account number when
withdrawing

Christian Grothoff

2016-03-31 17:45

manager   ~0010346

I'm not sure I get this. I thought we run all services under 'https' right now. Also, I don't know why we have any limit on the number of characters for any URL. That makes little sense.

My best guess is that there is a 'hack' in which username=URL in the bank, and because the wallet sends usernames instead of account numbers something worked with http but not https (as username doesn't exist and would for https be too long). Then indeed the right fix is to quickly fix the wallet to properly send the JSON with the Wire transfer data (i.e. the exchange's account number, instead of / in addition to its name).

Marcello Stanisci

2016-04-01 08:59

reporter   ~0010347

Yes, your guess is right, and the issue related to this one is the wallet's fix
for this scenario

Torsten Grothoff

2016-04-02 11:53

reporter   ~0010354

is it fixed?

Marcello Stanisci

2016-04-06 11:36

reporter   ~0010389

yes!

Issue History

Date ModifiedUsernameFieldChange
2016-03-23 18:33Torsten GrothoffNew Issue
2016-03-23 18:33Torsten GrothoffStatusnew => assigned
2016-03-23 18:33Torsten GrothoffAssigned To => Florian Dold
2016-03-29 13:24Christian GrothoffProduct Version => 0.0
2016-03-29 13:24Christian GrothoffTarget Version => 0.0
2016-03-31 14:52Marcello StanisciAssigned ToFlorian Dold => Marcello Stanisci
2016-03-31 14:55Marcello StanisciNote Added: 0010344
2016-03-31 15:55Marcello StanisciRelationship addedrelated to 0004354
2016-03-31 17:45Christian GrothoffNote Added: 0010346
2016-04-01 08:59Marcello StanisciNote Added: 0010347
2016-04-01 09:12Christian GrothoffAssigned ToMarcello Stanisci => Florian Dold
2016-04-02 11:53Torsten GrothoffNote Added: 0010354
2016-04-02 11:58Torsten GrothoffTag Attached: strethyfghtrdyou
2016-04-05 23:38Florian DoldAssigned ToFlorian Dold => Marcello Stanisci
2016-04-06 11:36Marcello StanisciNote Added: 0010389
2016-04-06 11:36Marcello StanisciStatusassigned => informing
2016-04-06 11:36Marcello StanisciResolutionopen => duplicate
2016-04-06 14:02Christian GrothoffStatusinforming => closed
2016-11-08 12:29Christian GrothoffTag Detached: strethyfghtrdyou
2023-04-13 18:37Florian DoldCategorywallet (WebExtensions) => wallet (WebExtension)