View Issue Details

IDProjectCategoryView StatusLast Update
0009549Talerwallet (all platforms)public2025-04-03 22:02
ReporterChristian Grothoff Assigned ToFlorian Dold  
PrioritynormalSeverityblockReproducibilityalways
Status feedbackResolutionopen 
Platformi7OSDebian GNU/LinuxOS Versionsqueeze
Product Versiongit (master) 
Target Version1.0 
Summary0009549: KYC-weirdness: wallet tries kyc-wallet for 1 CHF
DescriptionI 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.
TagsNo tags attached.

Activities

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

sebasjm

2025-04-03 21:35

developer   ~0024393

jq '.databases["taler-wallet-main-v10"].stores.transactions.records[0].value.transactionItem.kycUrl' is NULL

the migration from 0.13.1 to 0.14.4 didn't take into accounts transactions in kyc states and the spec for the URL changed.

sebasjm

2025-04-03 21:53

developer   ~0024394

but
jq '.databases["taler-wallet-main-v10"].stores.withdrawalGroups.records[1].value.kycAccessToken' has the kycToken present.
Why do we have kycUrl and accesToken? is kycUrl deprecated?

sebasjm

2025-04-03 22:01

developer   ~0024395

withdrawal status in the db
$ printf "%x\n" 16777222
1000006

-> PendingBalanceKyc

and wallet core is only building the URL for PendingKyc and SuspendedKyc
https://git.taler.net/taler-typescript-core.git/tree/packages/taler-wallet-core/src/withdraw.ts?id=07fc66eff79b8cc8e2325e56ccd269ae25e7a394#n384

sebasjm

2025-04-03 22:01

developer   ~0024396

07fc66eff..a6d2437eb

adds missing status

Issue History

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-21 16:49 Christian Grothoff Assigned To Christian Grothoff => sebasjm
2025-02-22 21:47 Christian Grothoff Priority urgent => normal
2025-03-21 01:59 Christian Grothoff Status feedback => assigned
2025-04-03 21:35 sebasjm Note Added: 0024393
2025-04-03 21:53 sebasjm Note Added: 0024394
2025-04-03 22:01 sebasjm Note Added: 0024395
2025-04-03 22:01 sebasjm Note Added: 0024396
2025-04-03 22:02 sebasjm Assigned To sebasjm => Florian Dold
2025-04-03 22:02 sebasjm Status assigned => feedback