View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0008079 | Taler | exchange | public | 2024-01-12 14:41 | 2024-08-28 21:31 |
| Reporter | Christian Grothoff | Assigned To | Florian Dold | Resolution deadline | |
| Implementer | Reviewer | ||||
| Release signed by | Deployed by | ||||
| Verifier | Communicator | ||||
| Data protection authority notification required | No | ||||
| CSIRT notification required | No | ||||
| Estimated effort (hours) | Actual effort (hours) | 0 | |||
| Fast-track | No | ||||
| Work package | none | ||||
| Priority | high | Severity | major | Reproducibility | always |
| Status | closed | Resolution | fixed | ||
| Platform | i7 | OS | Debian GNU/Linux | OS Version | squeeze |
| Product Version | git (master) | ||||
| Target Version | 0.13 | Fixed in Version | 0.13 | ||
| Summary | 0008079: cannot restart KYC process after KYC data was declined by KYC provider [4h] | ||||
| Description | ... also, exchange still sees the process as not 'finished' in the database, even though the KYC provider is on the final 'rejected' state (seen with KYCAID). | ||||
| Tags | No tags attached. | ||||
| ID | Summary | Status | Assigned To | Estimated effort (hours) |
|---|---|---|---|---|
| 0008708 | wallet-core should poll on KYC status [2h] | closed | Florian Dold | |
| This issue (hours) | ||||
| Its children (hours) | 0 | |||
| Together (hours) | 0 | |||
Nothing is missing: the plan can be finished.
| parent of | 0008708 | closed | Florian Dold | wallet-core should poll on KYC status [2h] |
| related to | 0008055 | closed | Christian Grothoff | 'finished' in legitimization_processes remains 'false' after KYC process is complete [2d] |
| child of | 0007681 | closed | Florian Dold | demo doesn't have any KYC checks [3d] |
| While 0008055 *might* have fixed this for KYCAID (still to be tested!), the issue is also very real with Challenger/OAuth2. Here, extra fun is that we get no notification of the ultimate failure state. Should discuss with Martin if we're doing this "wrong" and/or how to do it right. | |
| Fixed challenger to provide us an update when the process failed. The exchange now properly re-starts a fresh KYC process when the previous one has ultimately failed. However, the *wallet-core* never refreshes the KYC link, so even if a KYC process has ultimately failed, the wallet will always provide the (stale) link to the original KYC process -- even when 'refresh' is explicitly selected. Wallet-core thus needs to (1) long-poll to learn about KYC status updates *and* (2) if it does get a new KYC URL, actually provide that new link to the GUIs. | |
| Note that in the new KYC design, wallet-core will NOT HAVE TO refresh the KYC link, as it will be *stable*. So this is now an exchange-issue, not a wallet-core issue. | |
| What about the long-polling? That's still relevant, no? | |
| Yes, long-polling you should do. | |
| Needs wallet to support new KYC process, and ideally wallet-harness test to see if this is still an issue... | |
| Looksike what's missing is a KYC integration test that uses forms instead of just an AMLO decision. | |
| Covered with $ taler-harness run-integrationtests kyc-form-withdrawal | |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2024-01-12 14:41 | Christian Grothoff | New Issue | |
| 2024-01-12 14:41 | Christian Grothoff | Status | new => assigned |
| 2024-01-12 14:41 | Christian Grothoff | Assigned To | => Christian Grothoff |
| 2024-01-12 15:54 | Christian Grothoff | Relationship added | related to 0008055 |
| 2024-01-18 22:38 | Christian Grothoff | Relationship added | child of 0007681 |
| 2024-02-06 00:36 | Christian Grothoff | Summary | cannot restart KYC process after KYC data was declined by KYC provider => cannot restart KYC process after KYC data was declined by KYC provider [2d] |
| 2024-02-14 13:31 | Christian Grothoff | Note Added: 0021292 | |
| 2024-02-18 12:23 | Christian Grothoff | Note Added: 0021339 | |
| 2024-02-18 12:23 | Christian Grothoff | Assigned To | Christian Grothoff => Florian Dold |
| 2024-02-18 22:07 | Christian Grothoff | Summary | cannot restart KYC process after KYC data was declined by KYC provider [2d] => cannot restart KYC process after KYC data was declined by KYC provider |
| 2024-04-07 18:29 | Florian Dold | Status | assigned => implementing |
| 2024-04-07 18:29 | Florian Dold | Relationship added | parent of 0008708 |
| 2024-04-07 18:31 | Christian Grothoff | Note Added: 0022136 | |
| 2024-04-07 18:31 | Christian Grothoff | Assigned To | Florian Dold => Christian Grothoff |
| 2024-04-07 18:31 | Christian Grothoff | Status | implementing => assigned |
| 2024-04-07 18:42 | Florian Dold | Note Added: 0022138 | |
| 2024-04-07 23:08 | Christian Grothoff | Note Added: 0022141 | |
| 2024-05-05 13:42 | Christian Grothoff | Target Version | 0.11 => 0.14 |
| 2024-05-13 19:19 | Christian Grothoff | Target Version | 0.14 => 0.13 |
| 2024-07-25 22:21 | Christian Grothoff | Summary | cannot restart KYC process after KYC data was declined by KYC provider => cannot restart KYC process after KYC data was declined by KYC provider [4h] |
| 2024-08-08 21:10 | Christian Grothoff | Assigned To | Christian Grothoff => Florian Dold |
| 2024-08-08 21:11 | Christian Grothoff | Note Added: 0022958 | |
| 2024-08-19 18:19 | Florian Dold | Note Added: 0023003 | |
| 2024-08-21 09:01 | Florian Dold | Status | assigned => informing |
| 2024-08-21 09:01 | Florian Dold | Resolution | open => fixed |
| 2024-08-21 09:01 | Florian Dold | Note Added: 0023016 | |
| 2024-08-26 20:29 | Christian Grothoff | Fixed in Version | => 0.13 |
| 2024-08-28 21:31 | Christian Grothoff | Status | informing => closed |