View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0009694 | Taler | wallet-core | public | 2025-04-03 18:32 | 2025-04-04 19:15 |
| Reporter | sebasjm | Assigned To | Florian Dold | ||
| Priority | high | Severity | major | Reproducibility | always |
| Status | resolved | Resolution | fixed | ||
| Product Version | git (master) | ||||
| Target Version | 1.0 | ||||
| Summary | 0009694: wallet core moves from pending-kyc to pending-withdrawing without interaction | ||||
| Description | ./bin/taler-harness.mjs run-integrationtests kyc .... .... wallet-core notification { type: 'transaction-state-transition', oldTxState: { major: 'pending', minor: 'withdraw-coins' }, newTxState: { major: 'pending', minor: 'kyc' }, transactionId: 'txn:withdrawal:CHRTPA9VZ64HFMFB8EDR036XX9PD2FFC6GS4186W0NSY72CV38ZG' } 2025-04-03T16:30:28.853Z test-kyc.ts INFO got kyc notification: [ "{\n \"type\": \"transaction-state-transition\",\n \"oldTxState\": {\n \"major\": \"pending\",\n \"minor\": \"withdraw-coins\"\n },\n \"newTxState\": {\n \"major\": \"pending\",\n \"minor\": \"kyc\"\n },\n \"transactionId\": \"txn:withdrawal:CHRTPA9VZ64HFMFB8EDR036XX9PD2FFC6GS4186W0NSY72CV38ZG\"\n}" ] wallet-core notification { type: 'balance-change', hintTransactionId: 'txn:withdrawal:CHRTPA9VZ64HFMFB8EDR036XX9PD2FFC6GS4186W0NSY72CV38ZG', isInternal: false } wallet-core notification { type: 'transaction-state-transition', oldTxState: { major: 'pending', minor: 'kyc' }, newTxState: { major: 'pending', minor: 'withdraw-coins' }, transactionId: 'txn:withdrawal:CHRTPA9VZ64HFMFB8EDR036XX9PD2FFC6GS4186W0NSY72CV38ZG' } .... ... FATAL: test failed with exception Error: unexpected notification { "type": "transaction-state-transition", "oldTxState": { "major": "pending", "minor": "kyc" }, "newTxState": { "major": "pending", "minor": "withdraw-coins" }, "transactionId": "txn:withdrawal:CHRTPA9VZ64HFMFB8EDR036XX9PD2FFC6GS4186W0NSY72CV38ZG" } | ||||
| Tags | No tags attached. | ||||
|
|
the check in the integration test is not optimal but works for this purpose |
|
|
Bug: The function `processWithdrawalGroupPendingKyc` is wrongly updating the variable `kycWithdrawalDelay` making the function `processWithdrawalGroupPendingReady` to pause without any action for too long Behavior: When a transaction is found to be in `PendingKyc` it immediately goes into `PendingReady` (wrong!). When the task triggers again the withdrawal status is not checked. Only after the delay the transaction is checked again, maybe to find out that the transaction is still in `pending(kyc)` and the loop starts again. Expected: When a transaction goes into `PendingKyc` for the first time it should also save a timeout variable (defined as the delay) to check if the operation is now possible even if the account still requires the KYC procedure. |
|
|
9b1186d642b447acab7dd321d46dbaa3654565d7 |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2025-04-03 18:32 | sebasjm | New Issue | |
| 2025-04-03 18:32 | sebasjm | Status | new => assigned |
| 2025-04-03 18:32 | sebasjm | Assigned To | => Florian Dold |
| 2025-04-03 18:32 | sebasjm | Note Added: 0024387 | |
| 2025-04-04 19:02 | sebasjm | Note Added: 0024407 | |
| 2025-04-04 19:15 | sebasjm | Status | assigned => resolved |
| 2025-04-04 19:15 | sebasjm | Resolution | open => fixed |
| 2025-04-04 19:15 | sebasjm | Note Added: 0024408 |
