View Issue Details
|ID||Project||Category||View Status||Date Submitted||Last Update|
|0005033||Taler||bank (demonstrator)||public||2017-05-24 23:53||2017-06-06 14:18|
|Reporter||Christian Grothoff||Assigned To||Marcello Stanisci|
|Platform||i7||OS||Debian GNU/Linux||OS Version||squeeze|
|Product Version||git (master)|
|Target Version||0.3||Fixed in Version||0.3|
|Summary||0005033: "invalid parameters" on withdraw|
|Description||When I try to withdraw from the bank (local setup on my system @ home), I get "invalid parameters" on the /pin/question-page. The error message is horrible as it doesn't say what parameters are invalid (and does not mention a globally unique error code either). Can't say if the bank or the wallet is to blame either. The information provided by the exchange seems correct (uses the correct bank account number and its correct URL at localhost:8081).|
|Additional Information||Request URI is:|
|Tags||No tags attached.|
More details should be returned about the failure, as of 9430850.
Unfortunately, I couldn't reproduce this error locally; that suggests you are
running some outdated code - some leftover from ancient installs.
Ah, check that the wallet is updated too.
The parameter 'wire_details' is bad, as it clearly lacks a field named "name".
This suggests that the wallet is old.
Hmm. Is 'name' mandatory? Because:
1) the exchange doesn't check for it;
2) the wallet version I have clearly doesn't complain if it is missing.
Should it be mandatory?
The wallet treats wire details _mostly_ as an opaque object, it just removes methods that are not in the wt_types given by the bank and deletes the fees/salt/sig properties, because otherwise the URL is too long and the fees are not used by the bank anyway.
Otherwise it doesn't do any parsing, and I'm not sure it should.
||Ok, so let's make 'name' optional. No need for excessive checks, and while 'name' clearly often makes sense, I don't see why we really need it.|
||It's optional starting from b3486f1. Please close this whenever you'll correctly withdraw.|
||Ok, this part works now (but see my e-mail)...|
|2017-05-24 23:53||Christian Grothoff||New Issue|
|2017-05-24 23:53||Christian Grothoff||Status||new => assigned|
|2017-05-24 23:53||Christian Grothoff||Assigned To||=> Marcello Stanisci|
|2017-05-24 23:54||Christian Grothoff||Relationship added||child of 0003464|
|2017-05-29 10:56||Marcello Stanisci||Note Added: 0012179|
|2017-05-29 13:36||Marcello Stanisci||Note Added: 0012181|
|2017-05-29 13:44||Christian Grothoff||Note Added: 0012183|
|2017-05-29 14:48||Florian Dold||Note Added: 0012184|
|2017-05-29 14:49||Florian Dold||Note Edited: 0012184||View Revisions|
|2017-05-29 14:56||Christian Grothoff||Note Added: 0012185|
|2017-05-29 15:03||Marcello Stanisci||Note Added: 0012186|
|2017-05-29 18:27||Christian Grothoff||Status||assigned => resolved|
|2017-05-29 18:27||Christian Grothoff||Resolution||open => fixed|
|2017-05-29 18:27||Christian Grothoff||Fixed in Version||=> 0.3|
|2017-05-29 18:27||Christian Grothoff||Note Added: 0012187|
|2017-06-06 14:18||Christian Grothoff||Status||resolved => closed|