Skip to content
This repository has been archived by the owner on Jan 12, 2019. It is now read-only.

Card information shown in the card.io logs #184

Closed
bande14 opened this issue Mar 14, 2017 · 3 comments · Fixed by card-io/card.io-Android-source#126
Closed

Card information shown in the card.io logs #184

bande14 opened this issue Mar 14, 2017 · 3 comments · Fixed by card-io/card.io-Android-source#126

Comments

@bande14
Copy link

bande14 commented Mar 14, 2017

General information

SDK/Library version: 5.4.0

compile 'io.card:android-sdk:5.4.0'

Issue description

Using an app with log disabled we can still see some logs reported by the card.io library with confidential information in it. Please take into account that the library shows the last four digits of the PAN, the payment scheme and - when successful - also the expiry date. We tried to remove such logs but we were not able - any hint?

See below for instance:

01-17 16:32:14.073 636-15231/? W/libsonylc: Invalid ConfigID 24 25
01-17 16:32:14.139 636-15231/? W/libsonylc: Invalid ConfigID 24 25
01-17 16:32:14.206 636-15231/? W/libsonylc: Invalid ConfigID 24 25
01-17 16:32:14.248 16039-16039/? D/CardScanner: detected card: {MasterCard: •••• •••• •••• 6658}
01-17 16:32:14.248 16039-16039/? D/CardIOActivity: onCardDetected()
01-17 16:32:14.250 16039-16039/? D/CardScanner: setFlashOn: false
01-17 16:32:14.273 636-15231/? W/libsonylc: Invalid ConfigID 24 25

Issue seen for all devices and Android OS used.

@lkorth
Copy link
Member

lkorth commented Mar 14, 2017

Logging this information is less than ideal. We'll make some changes to remove the logging in the next version.

lkorth added a commit to card-io/card.io-Android-source that referenced this issue Mar 14, 2017
Stop logging the scanned card, fixes card-io/card.io-Android-SDK#184
lkorth added a commit to card-io/card.io-Android-source that referenced this issue Mar 17, 2017
Stop logging the scanned card, fixes card-io/card.io-Android-SDK#184
@lkorth
Copy link
Member

lkorth commented Mar 17, 2017

The fix for this was just released in 5.5.1

@bande14
Copy link
Author

bande14 commented Mar 20, 2017

Thank you guys!

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

Successfully merging a pull request may close this issue.

2 participants