0009549: KYC-weirdness: wallet tries kyc-wallet for 1 CHF

ID	Project	Category	View Status	Date Submitted	Last Update

0009549	Taler	wallet (all platforms)	public	2025-02-17 08:54	2025-02-21 16:49

Reporter	Christian Grothoff	Assigned To	sebasjm
Priority	urgent	Severity	block	Reproducibility	always
Status	feedback	Resolution	open
Platform	i7	OS	Debian GNU/Linux	OS Version	squeeze
Product Version	git (master)
Target Version	1.0

Summary	0009549: KYC-weirdness: wallet tries kyc-wallet for 1 CHF
Description	I still cannot withdraw from the TOPS exchange. There are several bugs with the KYC process: 1) While /keys clearly states no applicable withdraw or balance limit, the wallet makes a request to /kyc-wallet for just 1 CHF (even though it should be withdrawing 2 CHF!). So the amount is wrong, and the request is also wrong as the public limits of the exchange are clearly higher. Furthermore, the wallet did not even attempt a /withdraw. 2) Incorrectly (IMO), the exchange responds with a 451, despite there not being a wallet balance limit. 3) The wallet then tries to GET /kyc-check/1 which fails with a 400 Bad request, the exchange claiming that a segment in the URL is malformed with respect to "h_payto". The exchange is correct, the spec says /kyc-check/$H_NORMALIZED_PAYTO, which "1" clearly is not.
Tags	No tags attached.

Christian Grothoff 2025-02-17 08:56 manager ~0024097	Note that this is with the FireFox WebExtension wallet, 0.13.1 1af7fbfb2d583984e4808f6cc52ad39ea8ef5c3d, which seems rather dated. Why do we have no more recent release?

Christian Grothoff 2025-02-17 09:06 manager ~0024098	Ok, I overlooked that the exchange configuration DOES have a CHF:1 limit for testing configured. So the wallet does OK to request /kyc-wallet, but then does the wrong GET /kyc-check/!

Florian Dold 2025-02-19 23:38 manager ~0024114	Uploaded the latest Webextension now. Please check if that changes anything for you. The wallet-core tests are all passing, so either we lack some important test or the wallet you tried was outdated.

Christian Grothoff 2025-02-20 09:02 manager ~0024115	I'm now on 0.14.4, and it shows "KYC check required" -- but gives me no link to begin the process! kyc-required.png (39,843 bytes) kyc-required.png (39,843 bytes)

Christian Grothoff 2025-02-20 09:03 manager ~0024116	When a withdraw hangs on KYC, the wallet obviously must allow me to open a tab/follow a link to the KYC SPA! WebExtension on withdraw doesn't do that!

sebasjm 2025-02-20 21:38 developer ~0024117	Can we have a dump of the wallet database? My guess is that the migration failed. Webex can do anything if wallet-core doesn't provide the kycURL. Here I think the problem is not in the webex

Christian Grothoff 2025-02-20 22:10 manager ~0024118	Voila. taler-wallet-database-2025_02_20_22_09.json.gz (2,429,123 bytes)

Date Modified	Username	Field	Change
2025-02-17 08:54	Christian Grothoff	New Issue
2025-02-17 08:54	Christian Grothoff	Status	new => assigned
2025-02-17 08:54	Christian Grothoff	Assigned To	=> Florian Dold
2025-02-17 08:55	Christian Grothoff	Description Updated
2025-02-17 08:56	Christian Grothoff	Note Added: 0024097
2025-02-17 08:56	Christian Grothoff	Assigned To	Florian Dold => sebasjm
2025-02-17 08:56	Christian Grothoff	Assigned To	sebasjm => Florian Dold
2025-02-17 09:06	Christian Grothoff	Note Added: 0024098
2025-02-19 23:37	Florian Dold	Assigned To	Florian Dold => Christian Grothoff
2025-02-19 23:38	Florian Dold	Note Added: 0024114
2025-02-20 09:02	Christian Grothoff	Note Added: 0024115
2025-02-20 09:02	Christian Grothoff	File Added: kyc-required.png
2025-02-20 09:02	Christian Grothoff	Assigned To	Christian Grothoff => sebasjm
2025-02-20 09:03	Christian Grothoff	Note Added: 0024116
2025-02-20 21:38	sebasjm	Note Added: 0024117
2025-02-20 21:38	sebasjm	Assigned To	sebasjm => Christian Grothoff
2025-02-20 21:38	sebasjm	Status	assigned => feedback
2025-02-20 22:10	Christian Grothoff	Note Added: 0024118
2025-02-20 22:10	Christian Grothoff	File Added: taler-wallet-database-2025_02_20_22_09.json.gz
2025-02-21 16:49	Christian Grothoff	Assigned To	Christian Grothoff => sebasjm