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

Set raw number #57

Open
micbenner opened this issue Aug 11, 2015 · 8 comments
Open

Set raw number #57

micbenner opened this issue Aug 11, 2015 · 8 comments
Assignees
Milestone

Comments

@micbenner
Copy link

While easily extendable to add, I think the CreditCard class should ship with a 'rawNumber' attribute, thereby allowed a credit card number to be set without the code stripping non-numeric characters.

The reason being that many providers, such as Eway, allow for credit card numbers to be encrypted before being sent. The fact that this field unknowingly removes non-numeric characters is quite a "gotcha"

@greydnls
Copy link
Contributor

The non-numeric characters are stripped for the Luhn validation. It might be a better idea to create a separate class such as EncryptedCreditCard that has the raw number as you suggested, and skips the luhn check.

Would that solve the problem that you're currently having?

@micbenner
Copy link
Author

This was awhile ago but it sounds like the perfect solution.

@greydnls greydnls added this to the V3 milestone Oct 13, 2015
@greydnls greydnls self-assigned this Oct 13, 2015
@greydnls
Copy link
Contributor

Awesome, I've got it marked for V3 release which will hopefully be coming in the near future.

@micbenner
Copy link
Author

Great to hear!

Thanks a bunch.

On Tue, Oct 13, 2015 at 5:42 PM, Kayla Daniels notifications@github.com
wrote:

Awesome, I've got it marked for V3 release which will hopefully be coming
in the near future.


Reply to this email directly or view it on GitHub
#57 (comment)
.

@SilverPreece
Copy link

V3 is out now - was this included? Client is asking if we can support this for PCI-DSS.

@barryvdh
Copy link
Member

I don't think so.

@judgej
Copy link
Member

judgej commented Jun 18, 2018

I tend to use the cardToken field to supply non-lunn strings that represent a card. Never done it for an encrypted card number though, normally it's a reference to a CC record (which usually includes all details entered for the card) temporarily stored at the gateway. Could an erupted card number arguably be a card token?

I'm just looking at a gateway that separately encrypts the CVV, expiry date and name too, all in individual fields, so it's going to be fun working out how that is going to fit in. We don't want to end up with rawExpiry, rawCvv, etc

@eileenmcnaughton
Copy link
Contributor

Having just dug into this with eway I think one design consideration is that the Omnipay plugin rather than the calling code should be making the decision about whether the card has been encrypted. The addition & success or otherwise of an encryption script is hard to detect in a gateway-agnostic way in the calling code - but pretty easy to check within the plugin as each Gateway has it's own conventions

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants