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

Switch not rejecting an invalid request received from DFSP during Transfer-Fulfillment stage #1547

Closed
6 tasks done
Dorota-MB opened this issue Aug 4, 2020 · 5 comments
Closed
6 tasks done
Assignees
Labels
oss-core This is an issue - story or epic related to a feature on a Mojaloop core service or related to it
Milestone

Comments

@Dorota-MB
Copy link

Dorota-MB commented Aug 4, 2020

Summary:

Forex Transfer failing at MTN but getting successful on Switch.

When SWITCH receives the PUT/Transfers request from payee DSFSP with transferState=RESERVED instead of transferState=COMMITTED (during the transfer fulfilment stage), SWITCH is not rejecting it and instead its responding back with a PUT /Transfer response with FSPIOP-Source as "switch" due to which the transaction is getting failed on payee DFSP end. but however the transaction is getting Successful (COMMITTED) on the SWITCH end although it received error 503 response from payee DFSP.

Observed behaviour:

SWITCH is NOT rejecting the Invalid REQUEST received from payee DFSP's during the Transfer-Fulfilment stage. The transaction is NOT getting ABORTED instead its getting COMMITTED on the SWITCH end.

Expected behaviour:

The transaction shouldn’t get COMMITTED,

Switch should just reject the invalid request received from payee DFSP and the transaction should timeout eventually. which is “ABORTED” and then both sides should get notifications.

Severity: Low

Priority: High

Steps to Reproduce:

Trigger the Forex transfer between the Sims one connecting to Mowali v1.0 and the other to Mowali v1.1

Transfer stage logs from fxp_adapter, fxp server and INT WSO2 are attached.

This happening in the COMMIT stage. “Step 10 - Commit payee Transfer” in the below pic:

image

Freshservice issue linked to Jira:
#SR-31463 (https://modusbox.freshservice.com/helpdesk/tickets/31463) - Switch NOT rejecting an Invalid REQUEST received from DFSP's during Transfer-Fulfilment stage

Component:
Version: Mowali release 13.1 - for OSS component versions see here https://modusbox.atlassian.net/wiki/spaces/CV/pages/780763358/Mowali+Release+13.1

Platform: SBX

Bug found/raised by: @Siva Chunduru

Notes:
Int_wso2_fspiop_sorce_issue_logs.txt
fxp_adapter_fspiop_source_issue_logs.txt
fxp_server_fspiop_source_issue_logs.txt

Pull Requests

@elnyry-sam-k elnyry-sam-k added bug oss-core This is an issue - story or epic related to a feature on a Mojaloop core service or related to it labels Aug 4, 2020
@oderayi oderayi self-assigned this Aug 10, 2020
@ramakrishnan-sankar ramakrishnan-sankar added this to the Sprint 11.2 milestone Aug 10, 2020
@oderayi
Copy link

oderayi commented Aug 11, 2020

Bug now fixed and released in Central Ledger v11.1.4

@oderayi oderayi closed this as completed Aug 11, 2020
@elnyry-sam-k
Copy link
Member

Thanks @oderayi , would be nice to have this verified on a deployment.. @amarmodus what do you think?

@elnyry-sam-k elnyry-sam-k reopened this Aug 11, 2020
@oderayi oderayi closed this as completed Aug 12, 2020
@amarmodus
Copy link

Thanks @oderayi , would be nice to have this verified on a deployment.. @amarmodus what do you think?

yes we should verify it @elnyry-sam-k

@oderayi
Copy link

oderayi commented Aug 12, 2020

Thanks @oderayi , would be nice to have this verified on a deployment.. @amarmodus what do you think?

Sure @elnyry-sam-k. I didn't realise you reopened it for deployment validation. I'll reopen with Review/QA status.

@oderayi oderayi reopened this Aug 12, 2020
@amarmodus amarmodus self-assigned this Aug 12, 2020
@amarmodus
Copy link

amarmodus commented Aug 14, 2020

Screenshot 2020-08-14 at 12 13 33

Screenshot 2020-08-14 at 12 13 53

--To be discussed--

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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

5 participants