-
Notifications
You must be signed in to change notification settings - Fork 15
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
In error notification to Payer FSP, header for source having wrong value #2557
Labels
Milestone
Comments
elnyry-sam-k
added
bug
Something isn't working or it has wrong behavior on a Mojaloop Core service
oss-core
This is an issue - story or epic related to a feature on a Mojaloop core service or related to it
labels
Oct 28, 2021
hi @sri-miriyala can you please confirm which version of central-ledger and ml-api-adapter this was observed on? |
elnyry-sam-k
changed the title
Error notification to Payer FSP header for source having wrong value
In error notification to Payer FSP, header for source having wrong value
Nov 5, 2021
mdebarros
added a commit
to mdebarros/ml-api-adapter
that referenced
this issue
Nov 8, 2021
… wrong value - fix for mojaloop/project#2557 - added new event type "ABORT_VALIDATION" branch to allow for notification handler to provide a correct destination header mapping when error when validation fails (i.e. invalid fulfilment)
mdebarros
added a commit
to mdebarros/central-ledger
that referenced
this issue
Nov 8, 2021
… wrong value - fix for mojaloop/project#2557 - Updated fulfil-handler logic to produce an action type "ABORT_VALIDATION" when a validation error occurs (i.e. invalid fulfilment) - updated util/scripts/populateTestData.sh script to create a default settlement model for local dev testing
mdebarros
added a commit
to mojaloop/central-services-shared
that referenced
this issue
Nov 8, 2021
… wrong value (#316) - fix for mojaloop/project#2557 - added new event type "ABORT_VALIDATION" to allow for fulfil handler to provide a meaningful error when validation fails (i.e. invalid fulfilment) - updated kafka mapping to support the above change - updated dependencies & fixed audit-resolve
Headers being returned by running the {
"request": {
"uniqueId": "1636447657836t78sjc",
"headers": {
"content-type": "application/vnd.interoperability.transfers+json;version=1.0",
"date": "Tue, 09 Nov 2021 08:47:37 GMT",
"authorization": "{$inputs.NORESPONSE_BEARER_TOKEN}",
"traceparent": "00-aabbb30ca88d7d0ded99b88cdd87124d-89bc6eb945b33b22-30",
"user-agent": "axios/0.21.1",
"connection": "keep-alive",
"fspiop-http-method": "PUT",
"fspiop-uri": "/transfers/12749fe3-028f-4f47-b349-af41cf0e0668/error",
"fspiop-source": "switch",
"fspiop-destination": "testingtoolkitdfsp",
"fspiop-signature": "{\"signature\":\"Wxi3q77uMClRNGgEiw7TkvP4RWiSqDNUWiNUZcmHdvtmvxWUlCewkivzFdWrY_fgnktaY9WVbsBrzIyRNyRTRNdlqFLEaWYAQ0KwztrAJAcBru1Xa9kszJJC7tWtQ2VztapGV8uJwhS8e_AXS5-bg7ttfmiTT9W14CO5SRXdwjtqpjBTzBn0lqvP1Jrh_iwrOTBAZW43N1qbIgnT9SuhiEXijQp6vOOAO3F9ZetIj_ZWwXs1pkl7fiVkLus-HIGXSx8vv43Sy6eMmhIpkvPxaNmck_wHUaOpJUxzCrk59QhF0_2AWf66yDiL7E-LaBbKKnQEjAZsPOLVozUOnYu_EQ\",\"protectedHeader\":\"eyJhbGciOiJSUzI1NiIsIkZTUElPUC1VUkkiOiIvdHJhbnNmZXJzLzEyNzQ5ZmUzLTAyOGYtNGY0Ny1iMzQ5LWFmNDFjZjBlMDY2OC9lcnJvciIsIkZTUElPUC1IVFRQLU1ldGhvZCI6IlBVVCIsIkZTUElPUC1Tb3VyY2UiOiJzd2l0Y2giLCJGU1BJT1AtRGVzdGluYXRpb24iOiJ0ZXN0aW5ndG9vbGtpdGRmc3AiLCJEYXRlIjoiVHVlLCAwOSBOb3YgMjAyMSAwODo0NzozNyBHTVQifQ\"}",
"tracestate": "mojaloop=eyJzcGFuSWQiOiI4OWJjNmViOTQ1YjMzYjIyIiwidGltZUFwaUZ1bGZpbCI6IjE2MzY0NDc2NTc3NDQifQ==",
"content-length": "108",
"host": "moja2-ml-testing-toolkit-backend:5000"
},
"queryParams": {},
"body": {
"errorInformation": {
"errorCode": "3100",
"errorDescription": "Generic validation error - invalid fulfilment"
}
}
}
} |
mdebarros
added a commit
to mojaloop/central-ledger
that referenced
this issue
Nov 10, 2021
… wrong value (#869) - fix for mojaloop/project#2557 - Updated fulfil-handler logic to produce an action type "ABORT_VALIDATION" when a validation error occurs (i.e. invalid fulfilment) - updated util/scripts/populateTestData.sh script to create a default settlement model for local dev testing - chore: updated dependencies - main update to central-services-shared - added get-port to ncurc ignore list due to the breaking changes it introduces - updated all other dependencies - fixed audit-resolve issues
mdebarros
added a commit
to mojaloop/ml-api-adapter
that referenced
this issue
Nov 11, 2021
… wrong value (#488) - fix for mojaloop/project#2557 - added new event type "ABORT_VALIDATION" branch to allow for notification handler to provide a correct destination header mapping when error when validation fails (i.e. invalid fulfilment) - added unit tests for processing "ABORT_VALIDATION" events - chore: updated dependencies - main update to central-services-shared - updated all other dependencies - fixed audit-resolve issues
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Summary:
When the payeefsp is sending an invalid fulfillment, the switch is responding with an error callback to payerfspwith FSPIOP-Source header with the "payee fsp's ID" instead of "switch".
Severity:
Medium
Priority:
Medium
Expected Behavior:
The error message to the payer fsp includes the switch as the header value for fspiop-source header for error messages that originate in the Switch
Steps to Reproduce
Specifications
Notes:
More details, log: mojaloop/central-ledger#868
The text was updated successfully, but these errors were encountered: