View Issue Details

IDProjectCategoryView StatusLast Update
0006427Talerlibeufin-bankpublic2023-09-23 15:09
ReporterMS Assigned ToMS  
PrioritylowSeverityfeatureReproducibilityhave not tried
Status closedResolutionfixed 
Target Version0.9.3Fixed in Version0.9.3 
Summary0006427: EBICS errors need XML
DescriptionIn 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 InformationMoving to 0.2 because it became a testing problem.
TagsNo tags attached.

Activities

MS

2020-07-23 13:42

reporter   ~0016507

Last edited: 2020-07-23 13:44

Mostly addressed here: c716d23073be8846d0765fa766de341dbc953a88

MS

2020-07-23 13:48

reporter   ~0016508

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.

MS

2020-07-23 17:13

reporter   ~0016509

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.

MS

2020-07-29 15:41

reporter   ~0016531

Fixed, but this test now fails: integration-tests/test-ebics-double-payment-submission.py

MS

2021-03-25 10:42

reporter   ~0017647

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.

Issue History

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