View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0010196 | Taler | merchant backend | public | 2025-07-16 21:00 | 2025-07-17 20:53 |
Reporter | Florian Dold | Assigned To | Christian Grothoff | ||
Priority | normal | Severity | minor | Reproducibility | have not tried |
Status | resolved | Resolution | fixed | ||
Product Version | 1.0 | ||||
Target Version | 1.0 stretch goals | Fixed in Version | 1.0 stretch goals | ||
Summary | 0010196: merchant backend returns KYC status for exchanges disabled in config | ||||
Description | ... this in turn leads the SPA to "crash" because the merchant violates the protocol. We have "status": "kyc-wire-required", but no "payto_kycauths" field. Instead, the merchant should not return the KYC status for disabled exchanges. | ||||
Additional Information | 2025-07-16T20:58:54.450666+0200 taler-merchant-httpd-178471(T9B0H9F0RDVAGXCSPXDTHDJECG) WARNING No keys, thus no default KYC limits known 2025-07-16T20:58:54.450772+0200 taler-merchant-httpd-178471(29MGAC751ZHQ00X3WA306PHCCM) WARNING Exchange `https://exchange.taler-ops.ch/' not configured 2025-07-16T20:58:54.450779+0200 taler-merchant-httpd-178471(29MGAC751ZHQ00X3WA306PHCCM) ERROR Assertion failed at taler-merchant-httpd_private-get-instances-ID-kyc.c:972. 2025-07-16T20:58:54.450787+0200 taler-merchant-httpd-178471(29MGAC751ZHQ00X3WA306PHCCM) WARNING No keys, thus no default KYC limits known | ||||
Tags | No tags attached. | ||||
|
Fix committed to master branch. |
|
Should be fixed in 8fc8be0d..e4f7102c (but did not test, still, the patch is trivial). |
merchant: master e4f7102c 2025-07-17 22:52 Details Diff |
fix 0010196: do not return KYC status for untrusted exchanges |
Affected Issues 0010196 |
|
mod - src/backend/taler-merchant-httpd_exchanges.c | Diff File | ||
mod - src/backend/taler-merchant-httpd_exchanges.h | Diff File | ||
mod - src/backend/taler-merchant-httpd_private-get-instances-ID-kyc.c | Diff File |
Date Modified | Username | Field | Change |
---|---|---|---|
2025-07-16 21:00 | Florian Dold | New Issue | |
2025-07-17 20:46 | Christian Grothoff | Assigned To | => Christian Grothoff |
2025-07-17 20:46 | Christian Grothoff | Status | new => assigned |
2025-07-17 20:52 | Christian Grothoff | Changeset attached | => merchant master e4f7102c |
2025-07-17 20:52 | Christian Grothoff | Note Added: 0025519 | |
2025-07-17 20:52 | Christian Grothoff | Status | assigned => resolved |
2025-07-17 20:52 | Christian Grothoff | Resolution | open => fixed |
2025-07-17 20:52 | Christian Grothoff | Fixed in Version | => 1.0 stretch goals |
2025-07-17 20:52 | Christian Grothoff | Note Added: 0025520 | |
2025-07-17 20:53 | Christian Grothoff | Product Version | => 1.0 |
2025-07-17 20:53 | Christian Grothoff | Target Version | 1.1 => 1.0 stretch goals |