Cybersource: Add apple_pay params for discover if flag passed #5213
Conversation
DustinHaefele
force-pushed
the
cybs-apple-pay-discover
branch
from
bd4d892
to
aeb07d4
Compare
|
|
||
| xml.tag! 'ccAuthService', { 'run' => 'true' } do | ||
| xml.tag!('cavv', payment_method.payment_cryptogram) unless commerce_indicator | ||
| xml.commerceIndicator commerce_indicator.nil? ? ECI_BRAND_MAPPING[brand] : commerce_indicator |
There was a problem hiding this comment.
Do we also want to pass options[:reconciliation_id]?
There was a problem hiding this comment.
No I don't think so. That is noted in the visa docs, but not the discover docs.
| return unless options[:enable_cybs_discover_apple_pay] | ||
|
|
||
| xml.tag! 'ccAuthService', { 'run' => 'true' } do | ||
| xml.tag!('cavv', payment_method.payment_cryptogram) unless commerce_indicator |
There was a problem hiding this comment.
with the underlying data, what will the commerce indicator actually be.. the non Network tokenization PB or the DIPB ?
There was a problem hiding this comment.
Based on line 911 the commerce_indicator will be internet in the case of network tokenization and it will be pb for non-network tokenization
There was a problem hiding this comment.
in the cybersource docs, its seems DIPB is what is required in this context
https://developer.cybersource.com/content/dam/docs/cybs/en-us/apple-pay/developer/smartpay/so/applepay.pdf
There was a problem hiding this comment.
Based on line 911 the commerce_indicator will be internet in the case of network tokenization and it will be pb for non-network tokenization
i think that code is only for SUBSEQUENT, when its merchant initiated follow up. as a way to do recurring.. i think the other code earlier (line 902) has the logic the other way 'internet' for regular, and 'pb' for network tokenization
There was a problem hiding this comment.
Yes the 902 code is for pure network tokens, and not related to apple_pay, that is differentiated in add_auth_service around line 795.
This is also a little confusing because in the example, as you posted, it shows dipb. But in the actual definition of required elements it says it should be internet. I think I am going to go with internet in this case since it is written more explicitly in the docs.
DustinHaefele
force-pushed
the
cybs-apple-pay-discover
branch
from
e00ac59
to
757cacd
Compare
DustinHaefele
force-pushed
the
cybs-apple-pay-discover
branch
from
757cacd
to
aaf9766
Compare
No description provided.