-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
CyberSource: include networkTokenCryptogram
in ccAuthService
for MC
#4834
base: master
Are you sure you want to change the base?
CyberSource: include networkTokenCryptogram
in ccAuthService
for MC
#4834
Conversation
See - https://developer.cybersource.com/docs/cybs/en-us/payments/developer/cielo/so/payments/payments-processing-basic-intro/pnt-auth-intro.html Schema: - https://developer.cybersource.com/library/documentation/dev_guides/Simple_Order_API_Clients/html/Topics/Using_XML1.htm - https://ics2ws.ic3.com/commerce/1.x/transactionProcessor/CyberSourceTransaction_1.211.xsd
@@ -813,6 +813,7 @@ def add_auth_network_tokenization(xml, payment_method, options) | |||
xml.tag!('collectionIndicator', DEFAULT_COLLECTION_INDICATOR) | |||
end | |||
xml.tag! 'ccAuthService', { 'run' => 'true' } do | |||
xml.tag!('networkTokenCryptogram', payment_method.payment_cryptogram) if payment_method.payment_cryptogram |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You mention that Visa requires this but the ccAuthService
tag is only used for Mastercards. Visa cards have the cryptogram added to the ccAuthService.cavv
field
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hello and TY @aenand! I corrected the description (Visa -> Cybersource). I meant to express that we got feedback, from Cybersource, asking that the network token cryptogram is provided under the ccAuthService.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah that makes more sense! I'll take a look at this Monday.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
TY! The PR is still in draft - I am confused by the fact the cryptogram we be provided twice in the same request.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah that is strange 🤔 do you have a contact to reach out to at CyberSource?
Marking this "of interest" before a cleanup of stale PRs |
Cybersource requires that the network token cryptogram is provided under the
ccAuthService
element. This PR adds it. Note this is redundant withauthenticationData
under theucaf
element.See Cybersource's Authorizations with Payment Network Tokens and CyberSourceTransaction_1.211.xsd, as per Cybersource's Using XML documentation.
Unit tests
Remote tests
Same outcome on
master
and on the PR's branch