Autofill Credit Card section could benefit from validation (length, content, etc)

[srirambv]

Did you search for similar issues before submitting this one?
Yes

Describe the issue you encountered:
Credit Card field should not take more than standard 19 characters. No check done for only numeric values.

Expected behavior:
Should only accept numbers and max length should be 19.

• Platform (Win7, 8, 10? macOS? Linux distro?):
  Windows 10 x64
• Brave Version:
  0.11.6 Beta 3
• Steps to reproduce:
  1. Open auttofill and add an entry under credit card
  2. Accepts more than 19 char in card field. Also able to enter char and save it. (Should not allow)
• Screenshot if needed:
  
• Any related issues:

[srirambv]

cc: @darkdh

[darkdh]

Could you separate the editing issue and update it to #3368? It's the same root cause and it might be better to leave field length issue here. Thanks.

[srirambv]

Done

[alexwykoff]

Perhaps we could concern ourselves with validating the number and tie the activation of the save control to a positive result?
https://gist.github.com/DiegoSalazar/4075533

[bsclifton]

@alexwykoff The validation would be a nice touch I created a separate issue for it so we can discuss more 😄
#3377

[darkdh]

I should mention that Chrome does not check the validation of the field.

I think it's better to save whatever user input for this milestone

[bridiver]

maybe move this into a general data validation ticket for another release? Chrome doesn't do any validation and I can enter letters into the CC field if I want

[bsclifton]

Updated title- this would be a great one to hear input from @bradleyrichter

This is a potential opportunity to give us an advantage over other browsers by performing validation and catching user errors with the information provided

Besides length check, we can also consider the approach @alexwykoff has mentioned of not allowing to save input (or giving a warning maybe) if the input doesn't pass a validation test. If we did that some questions:

• What should the warning message(s) look like?
• Should the validation be toggle-able, for getting around edge cases or if it's bothersome for the user?

More information which can be used for validation can be found here:
https://en.wikipedia.org/wiki/Payment_card_number

Maybe as a separate issue, we can alert folks on the Autofill screen if their card is nearing (or has passed) expiration

[bridiver]

I think it's a good idea, just wanted to move out of 0.11.6

[bsclifton]

Totally agree- Since milestone was removed, I closed #3377 and will put the details in the post above 😄 (ignore the accidental re-tag below, I was trying to view the milestone screen)

[bradleyrichter]

Checkbox idea is good here since we don't yet know of reasons why validation may be problematic. [x] "validate card number as I type"

Regarding improvements over the competition, (thanks @bsclifton ) how about showing the user which sites contain the expiring credit card number ahead of time so they can update them all with a low pain threshold before auto-payments fail? (assuming this is not a privacy concern.)

[srirambv]

Would it be possible to show logo/name of the card provider next to the card number. When you have multiple card details stored the dropdown looks very empty with just the ...card number

[bsclifton]

@srirambv great suggestion! I'm not sure if there are legal requirements around the logo and how it's displayed, but showing a logo is possible for sure

cc: @bradleyrichter

[bradleyrichter]

SGTM - here are the fontawesome icons:

[bridiver]

Actually I'm pretty sure those are already available from the auto fill client and we just need to use them

On Sep 6, 2016, at 3:50 PM, Brad Richter notifications@github.com wrote:

SGTM - here are the fontawesome icons:

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

[MargarytaChepiga]

I would love to tackle this!
As I understand we still want to do a validation for the credit card ( at least min and max characters? ). But what about the Credit card name and expiry dates?

[MargarytaChepiga]

When I was trying to reproduce the issue, I noticed the following:

So I have been looking into the code to see where we are adding the credit card and I found something interesting. I suppose that the following function is the one which executes when we don't have any credit card and we want to add one:

browser-laptop/js/about/autofill.js

Line 150 in 5143bd3

onAddCreditCard () {

and we are calling it right here:

browser-laptop/js/about/autofill.js

Line 234 in 5143bd3

onClick={this.onAddCreditCard}

However, I am not sure where do we actually save the credit card. Cause once I add it, it seems like the following get's executed too:

browser-laptop/js/about/autofill.js

Line 85 in 5143bd3

render () {

So I am not quite sure where to do the validation. But as I understand that's where we saving the credit card:

browser-laptop/app/renderer/components/autofill/autofillCreditCardPanel.js

Line 66 in 5143bd3

onSave () {

browser-laptop/js/actions/appActions.js

Line 662 in 5143bd3

addAutofillCreditCard: function (detail) {

[bsclifton]

@MargarytaChepiga one approach could be to disable the save button until the number provided is valid. You could then run validation on keydown / keyup for the text field (and also with expiration)

In that same code, if you have enough to detect which type of card it is (ex: it's valid and starts with 4, it's a VISA) you could then consider showing an icon next to it. As soon as the field becomes invalid, hide any icon

Another approach might be to look for an existing credit card React component that we can integrate with (something which may be on npm). We'd need to open a security review either way before accepting this, but it might be that someone has already solved this problem and has support for cards used internationally

[MargarytaChepiga]

Thank you so much @bsclifton for clarifying things for me! 😃
So far I have completed a simple length and numeric validation for credit card number:

Where length should be between 13 and 19 characters (according to this credit card validator).
I can also detect a credit card type, for example:

Where the UI change is only for the debugging purposes 😅
However, I haven't got to the icon part yet.

I also looked up for some React components, some of them are really cool and it looks like we could consider using them. Please take a look at these ones, and let me know what you think:

• Credit Card Validation

• Credit Card Validator

• This one looks really great, but as I understand it provides more of UI rather than validation ( has an icon though 😅 )

• Update: Last one accepts only up to 16 characters for the credit card number, which won't work for some of the credit cards.

[bsclifton]

I feel safe saying that we won't be able to address this with browser-laptop. Closing as wontfix in favor of attempted a fix in Brave Core:
brave/brave-browser#369