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

Credit card numbers are improperly formatted #11977

Open
1 task done
warsaw opened this issue Nov 13, 2024 · 6 comments
Open
1 task done

Credit card numbers are improperly formatted #11977

warsaw opened this issue Nov 13, 2024 · 6 comments
Labels
bug desktop Desktop Application

Comments

@warsaw
Copy link

warsaw commented Nov 13, 2024

Steps To Reproduce

  1. Go to 'Card'
  2. Click on a Visa or Mastercard
  3. View the Number

Expected Result

I'd expect to see a number formatted like 1234 5678 9012 3456, i.e. four groups of 4 numbers.

Actual Result

I see a number like 1234 567 8 90 12 3 456. This happens even if edit the card, delete the spaces and add them back in the right place.

Screenshots or Videos

Unavailable due to sensitive nature of actual numbers.

Additional Context

This doesn't happen for all cards. Some format just fine. I've only seen this for Visa and Mastercard numbers, but some Visa and Mastercard fields are fine. There's no commonality that I can tell. Only the macOS desktop version seems to be affected. iOS and iPadOS versions seem fine.

Operating System

macOS

Operating System Version

14.7.1

Installation method

Mac App Store

Build Version

2024.10.2 (32149)

Issue Tracking Info

  • I understand that work is tracked outside of GitHub. A PR will be linked to this issue should one be opened to address it, but Bitwarden doesn't use fields like "assigned", "milestone", or "project" to track progress.
@warsaw warsaw added bug desktop Desktop Application labels Nov 13, 2024
@bitwarden-bot
Copy link

Thank you for reporting this issue! We've added this to our internal tracking system.
ID: PM-14883

@rmcdowell-bitwarden
Copy link
Collaborator

Hi there,

Thank you for your report!

I was able to reproduce this issue, and I have flagged this to our engineering team.

If you wish to add any further information/screenshots/recordings etc., please feel free to do so at any time - our engineering team will be happy to review these.

Thanks once again!

@djsmith85
Copy link
Contributor

Hi @warsaw and thank you for your report.

While fixing #12505 for the new UI's I did also check the desktop client and the spacing appears correct.

The spacing is defined within credit-card-number.pipe.ts and supports a few common brands.

  • Is your credit-card maybe from an unsupported brand?
  • Or does the number not fit the specified pattern?

Kind regards,
Daniel

@djsmith85 djsmith85 added the needs-reply Closes issue after 14 days if no reply from OP label Dec 20, 2024
@warsaw
Copy link
Author

warsaw commented Dec 23, 2024

Hi @djsmith85

No, there's nothing odd or nonstandard about the cards I'm seeing the problem with. A perfect example is an Apple Card, brand Mastercard. On iOS and iPadOS Bitwarden, when you toggle reveal, the number shows up perfectly well as four four-digit spans separated by spaces. On macOS with Bitwarden client 2024.12.1 (34750) updated from the App Store, it shows up as 1234 567 8 90 12 3 456. Verified on both multiple macOS machines (both using Bitwarden from the App Store). I even tried editing the entry, completely deleting the number and retyping it as four four-digit spans, but again, it still shows up as above. I even tried sending the entire entry to the trash and entering an entirely new item information, and that still shows the card number incorrectly. Of course, syncing the vault on macOS and then on iOS shows it correctly on iOS.

@warsaw
Copy link
Author

warsaw commented Dec 23, 2024

Also, looking at the code you referenced, I don't see the block layout I'm seeing described in that record at all.

@warsaw
Copy link
Author

warsaw commented Dec 23, 2024

One other interesting data point. If you hit the copy icon on a badly formatted number in the macOS client, then paste the number, you do seem to get a correctly formatted 4x4 block of numbers. That in its own right can be problematic since not every website accepts spaces between blocks, but it's still interesting that even though the number is incorrectly formatted in the UI, it still at least pastes correctly.

@github-actions github-actions bot removed the needs-reply Closes issue after 14 days if no reply from OP label Dec 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug desktop Desktop Application
Projects
None yet
Development

No branches or pull requests

4 participants