View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0009719 | Taler | merchant backend | public | 2025-04-09 17:21 | 2025-04-11 22:07 |
Reporter | Christian Grothoff | Assigned To | Christian Grothoff | ||
Priority | normal | Severity | feature | Reproducibility | N/A |
Status | assigned | Resolution | open | ||
Platform | i7 | OS | Debian GNU/Linux | OS Version | squeeze |
Product Version | git (master) | ||||
Target Version | 1.0 | ||||
Summary | 0009719: test 0-limit for deposit instant-triggers KYC requirement in SPA | ||||
Description | This is the common use-case for GLS, and must work. | ||||
Tags | No tags attached. | ||||
|
test-kyc-merchant-deposit already validate this, but AFAIK to get an up to date information from the merchant backend the merchant needs to try to create an order, otherwise is just cached in the database |
|
kyc-merchant-deposit-rewrite has been updated to fail Ends with FATAL: test failed with exception Error: Too many retries here https://git.taler.net/taler-typescript-core.git/tree/packages/taler-harness/src/integrationtests/test-kyc-merchant-deposit-rewrite.ts#n241 After retrying 10 times asking kyc status |
|
Well we see the following in the logs: 2025-04-11T20:57:15.490909+0200 taler-merchant-kyccheck-46502 INFO Will repeat inquiry in 6 days That's after we got the KYC-still-required from the exchange. Naturally, your test doesn't then wait for 6 days. There are several points to consider here thus: 1) 6 days is probably a bit high; 2) the test probably should still not rely on the background job to do this in a timely fashion, and instead should start taler-merchant-kyccheck explicitly with timetravel (--timetravel) to force it to fetch the latest status. I'll go and reduce it from 6 days to 6 hours, but please add the taler-merchant-kyccheck with timetravel (after the KYC was done!). Let's see if we then pass. |
|
ee77fb01..ce5fe921 increases kyc-check frequency to 6h as long as we have relevant soft zero-limits in the limits. |
|
the test now does a timetravel +7 days before asking for kyc. still the same behavior |
Date Modified | Username | Field | Change |
---|---|---|---|
2025-04-09 17:21 | Christian Grothoff | New Issue | |
2025-04-09 17:21 | Christian Grothoff | Status | new => assigned |
2025-04-09 17:21 | Christian Grothoff | Assigned To | => sebasjm |
2025-04-10 22:24 | sebasjm | Note Added: 0024504 | |
2025-04-11 17:11 | sebasjm | Note Added: 0024513 | |
2025-04-11 17:12 | sebasjm | Assigned To | sebasjm => Christian Grothoff |
2025-04-11 17:12 | sebasjm | Category | merchant backoffice SPA => merchant backend |
2025-04-11 21:07 | Christian Grothoff | Note Added: 0024517 | |
2025-04-11 21:15 | Christian Grothoff | Note Added: 0024518 | |
2025-04-11 21:15 | Christian Grothoff | Assigned To | Christian Grothoff => sebasjm |
2025-04-11 22:07 | sebasjm | Note Added: 0024520 | |
2025-04-11 22:07 | sebasjm | Assigned To | sebasjm => Christian Grothoff |