View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0006427 | Taler | libeufin-bank | public | 2020-07-09 19:41 | 2023-09-23 15:09 |
| Reporter | MS | Assigned To | MS | ||
| Priority | low | Severity | feature | Reproducibility | have not tried |
| Status | closed | Resolution | fixed | ||
| Target Version | 0.9.3 | Fixed in Version | 0.9.3 | ||
| Summary | 0006427: EBICS errors need XML | ||||
| Description | In some (all?) error cases related to EBICS, the sandbox responds with plain text. Instead it should respond with the XML document that complies to the EBICS specifications. | ||||
| Additional Information | Moving to 0.2 because it became a testing problem. | ||||
| Tags | No tags attached. | ||||
|
|
Mostly addressed here: c716d23073be8846d0765fa766de341dbc953a88 |
|
|
The EbicsError global catcher needs a private key to be included in the exception object, and if that's not found, it responds with JSON (and status 500). (The private key is needed to sign the response object.) The current problem is that not in all the parts of the code such private key is included in the exception object, resulting still in JSON+500 being returned. |
|
|
After private discussion, the private key will be moved away from the exception object. But at the same time, the exception handler must be moved in a place where it is easier to get more context (including the private key) from the current request. |
|
|
Fixed, but this test now fails: integration-tests/test-ebics-double-payment-submission.py |
|
|
The test mentioned in the last comment was long ago removed, and in general, corner cases will 'all' have to be implemented in the new TypeScript harness. IMO, this can be closed. |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2020-07-09 19:41 | MS | New Issue | |
| 2020-07-09 19:45 | MS | Description Updated | |
| 2020-07-23 13:42 | MS | Note Added: 0016507 | |
| 2020-07-23 13:44 | MS | Note Edited: 0016507 | |
| 2020-07-23 13:48 | MS | Note Added: 0016508 | |
| 2020-07-23 17:13 | MS | Note Added: 0016509 | |
| 2020-07-29 15:41 | MS | Note Added: 0016531 | |
| 2020-07-29 15:41 | MS | Assigned To | => MS |
| 2020-07-29 15:41 | MS | Status | new => assigned |
| 2020-11-10 12:15 | MS | Target Version | => 0.9.1 |
| 2020-11-10 12:15 | MS | Additional Information Updated | |
| 2021-03-25 10:42 | MS | Note Added: 0017647 | |
| 2021-03-25 10:42 | MS | Priority | low => none |
| 2023-01-08 10:36 | Christian Grothoff | Severity | minor => feature |
| 2023-01-08 10:36 | Christian Grothoff | Product Version | => Git master |
| 2023-01-08 10:36 | Christian Grothoff | Target Version | 0.9.1 => 0.9.3 |
| 2023-04-11 11:33 | MS | Status | assigned => feedback |
| 2023-04-11 11:34 | MS | Priority | none => low |
| 2023-04-11 11:34 | MS | Target Version | 0.9.3 => |
| 2023-04-13 20:26 | Florian Dold | Category | sandbox => libeufin sandbox |
| 2023-04-13 20:38 | Florian Dold | Category | libeufin sandbox => libeufin-sandbox |
| 2023-04-13 20:50 | Florian Dold | Project | libeufin => Taler |
| 2023-04-17 11:04 | Florian Dold | Product Version | Git master => |
| 2023-04-17 11:04 | Florian Dold | Target Version | => 0.9.3 |
| 2023-06-29 11:59 | MS | Status | feedback => resolved |
| 2023-06-29 11:59 | MS | Resolution | open => fixed |
| 2023-09-23 15:07 | Christian Grothoff | Fixed in Version | => 0.9.3 |
| 2023-09-23 15:09 | Christian Grothoff | Status | resolved => closed |
| 2023-09-23 15:26 | Christian Grothoff | Category | libeufin-sandbox => libeufin-bank |
