Skip to content
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

Closed
elnyry-sam-k opened this issue Oct 28, 2021 · 2 comments
Closed
Assignees
Labels
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
Milestone

Comments

@elnyry-sam-k
Copy link
Member

elnyry-sam-k commented Oct 28, 2021

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

  1. Initiate a transfer ( parties, quotes calls and then POST /transfers)
  2. In the PUT /transfers/{ID} the fulfil request by payee fsp, send an invalid value (that passes schema validation)
  3. Notice the error message sent by the Switch to the Payer FSP

Specifications

  • Component (if known): central ledger
  • Version: Mojaloop v13.0.2
  • Platform: Any
  • Subsystem: notification handler (possibly fulfil handler)
  • Type of testing: Reported by an implementation team
  • Bug found/raised by: found & reported by @sri-miriyala

Notes:
More details, log: mojaloop/central-ledger#868

  • Severity when opened: Medium
  • Priority when opened: Medium
@elnyry-sam-k 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
@elnyry-sam-k elnyry-sam-k added this to the Sprint 16.1 milestone Nov 5, 2021
@elnyry-sam-k
Copy link
Member Author

hi @sri-miriyala can you please confirm which version of central-ledger and ml-api-adapter this was observed on?

@elnyry-sam-k 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
@mdebarros
Copy link
Member

Headers being returned by running the payee_invalid_fulfilment test case from the GP Suite:

{
  "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
Labels
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
Projects
None yet
Development

No branches or pull requests

2 participants