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

fix(#2557): error notification to payer fsp, header for source having wrong value #869

Conversation

mdebarros
Copy link
Member

@mdebarros mdebarros commented Nov 8, 2021

  • fix for In error notification to Payer FSP, header for source having wrong value 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

… 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 mdebarros self-assigned this Nov 8, 2021
- 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
@lewisdaly
Copy link
Contributor

the mpath vulnerability is marked as critical, so we probably shouldn't ignore it.

This PR: mojaloop/object-store-lib#33 will fix it

@mdebarros
Copy link
Member Author

mdebarros commented Nov 10, 2021

the mpath vulnerability is marked as critical, so we probably shouldn't ignore it.

This PR: mojaloop/object-store-lib#33 will fix it

I have a bigger concern, I am not sure anybody is even using bulk functionality, and thus it's not being maintained.

@lewisdaly
Copy link
Contributor

the mpath vulnerability is marked as critical, so we probably shouldn't ignore it

Yeah I can cover this in an upcoming PR

I have a bigger concern, I am not sure anybody is even using bulk functionality, and thus it's not being maintained.

Yeah that's true as well... but don't we have tests for it?

@lewisdaly
Copy link
Contributor

Also, how did this not break any tests?

@mdebarros mdebarros merged commit 472fc12 into mojaloop:master Nov 10, 2021
@mdebarros
Copy link
Member Author

mdebarros commented Nov 10, 2021

the mpath vulnerability is marked as critical, so we probably shouldn't ignore it

Yeah I can cover this in an upcoming PR

I have a bigger concern, I am not sure anybody is even using bulk functionality, and thus it's not being maintained.

Yeah that's true as well... but don't we have tests for it?

@elnyry-sam-k please feel free to expand, or give your view on this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants