Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

Pass Gateway response if Issuer response is empty#15

Open
fpolowood wants to merge 17 commits intohps:masterfrom
fpolowood:pass-gtw-response
Open

Pass Gateway response if Issuer response is empty#15
fpolowood wants to merge 17 commits intohps:masterfrom
fpolowood:pass-gtw-response

Conversation

@fpolowood
Copy link

When 'list'ing the Issuer response is always favored over the Gateway response, but in the case of Capture, the Issuer is empty and Gateway is relevant. So, if Issuer is empty, return Gateway, both for the code and the text.

@fpolowood
Copy link
Author

Fixes #16

@slogsdon
Copy link
Contributor

Hi @fpolowood! Apologies for the delay, and thanks for the contribution. We'll get this reviewed and will follow up as soon as we can.

Copy link
Contributor

@slogsdon slogsdon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Again, apologies for the delay. We're going to attempt to get this PR released next Tuesday, so if you're unable to address the requested changes, please let us know so we can tackle them on our end.

Thanks!

tx.CardData = {};
tx.CardData.ManualEntry = hlp.hydrateCardData(card);
if (typeof(card)==='string'){
tx.CardData.TrackData = card;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For track data, we'll need to have the ability to set the method attribute of the TrackData data field as either swipe or proximity to allow for contactless as well, so it may be better to pass the track data as an object similar to our other SDKs. You can reference our .NET SDK: Data Class, Hydration.

Also, our gateway supports encrypted track data (a large portion of the readers are encrypted with Heartland's End-To-End Encryption [E3]), so it would be nice to have this capability as well. You can reference our .NET SDK: Data Class, Hydration.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants