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

Expose CVC number in CardField when dangerouslyGetFullCardDetails=true #903

Closed
mashaalmemon opened this issue Apr 22, 2022 · 5 comments · Fixed by #928
Closed

Expose CVC number in CardField when dangerouslyGetFullCardDetails=true #903

mashaalmemon opened this issue Apr 22, 2022 · 5 comments · Fixed by #928
Labels
enhancement New feature or request

Comments

@mashaalmemon
Copy link

Is your feature request related to a problem? Please describe.

We use Stripe to take card payments, and a 2nd payment provider to handle Visa Debit payments. We want to use the Stripe CardField component to collect card details & submit them to both Stripe and the Visa Debit payment provider at the same time. In our use case our Visa Debit payment provider provides a client side library which we use to tokenize the payment method before adding it to their system.

We currently are using dangerouslyGetFullCardDetails on the CardField component to get number, expiryYear expiryMonth from the control but note that cvc is not provided. We need cvc to tokenize the information.

Describe the solution you'd like

We would appreciate it if cvc could be exposed as well when dangerouslyGetFullCardDetails=true on the CardField component.

Describe alternatives you've considered

Any alternative would involve using a different component or building our own component for Visa Debit information. This would not be ideal.

Additional context

In #280 access to the full card number was granted. In that ticket, providing cvc was considered by @thorsten-stripe but the original requestor there did not require it. This has been requested in and #425, but the use case was not appropriate. I believe @thorsten-stripe was looking for more information on a usecase but one wasn't identified. I hope our use-case justifies the need for this feature.

@charliecruzan-stripe charliecruzan-stripe added the enhancement New feature or request label Apr 25, 2022
@mashaalmemon
Copy link
Author

Hi @charliecruzan-stripe, any change this type of enhancement can be made. This seems to be the only piece of information that doesn't make it though when using dangerouslyGetFullCardDetails=true. Would be great if this could get in.

If we could get an idea if our request would be possible sooner-than-later we can make a decision what we should do on this end; We're currently marching towards a deadline on this end and this issue would need to be solved for our release to be feature complete.

Essentially, on our end if we can't get at the cvc value from the CardField component, we'll need to create or own component to collect visa debit information.

Let us know either way. Thanks in advance.

@mashaalmemon
Copy link
Author

@charliecruzan-stripe please advise is this request might make it into the subsequent release. If not we'll need to find another solution.

@charliecruzan-stripe
Copy link
Collaborator

hi @mashaalmemon , I'm checking in on this now. It's a fairly specific use case so I can't make any definitive promises right now unfortunately

@charliecruzan-stripe
Copy link
Collaborator

charliecruzan-stripe commented May 5, 2022

@mashaalmemon we will implement this, it will likely be ready for our next release. Please remember that enabling dangerouslyGetFullCardDetails means you are no longer eligible for an SAQ A. See more info here- https://stripe.com/docs/security/guide

@mashaalmemon
Copy link
Author

@charliecruzan-stripe, I appreciate the super quick turn around time. Regarding the SAQ A understood. For planning purposes, when do you expect the next release to be out/available?

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

Successfully merging a pull request may close this issue.

2 participants