microg / GmsCore

Free implementation of Play Services
https://microg.org
Apache License 2.0
7.7k stars 1.62k forks source link

[de.c24.bankapp] QR code for Online Banking feature in C24 Bank app not scannable #1637

Open matterharz opened 2 years ago

matterharz commented 2 years ago

Affected app Name: C24 Bank Package id: de.c24.bankapp

Describe the bug The QR code that must be scanned to access the nice-to-have Online Banking feature isn't recognized at all by the built-in scanner. ~There's no workaround since other scanners (like Binary Eye) that scan the code without any problem cannot share it with the app.~

To Reproduce

  1. Log In
  2. Tap on Profile, then Online Banking
  3. Try to scan the qr code at banking.c24.de
  4. Nothing happens (Yes, I've tried different monitor brightness settings, turned off Night Mode, etc.)

Expected behavior The 6 digits needed to access the online banking web frontend should appear on screen. (It's non-essential since this is a smartphone bank but nevertheless...). It does work with GApps, last tested in spring 2021, I think.

System Custom ROM: LineageOS for microG 18.1

microG microG Core version: 0.2.22.212658 microG Self-Check results: All ticked

matterharz commented 2 years ago

No change using 0.2.24.214816.

j4ncp commented 2 years ago

Can confirm this, that bug affects me as well.

I can contribute that it seems to be trying to use the com.google.android.gms.vision.barcode API, which was supposedly implemented/fixed in microg in #670, however, the scanning does indeed not work, just as described by @matterharz. I can provide a logcat: logcat_c24app.txt

DrBarns commented 5 months ago

Can confirm this too, after recent update to Version 2.73.1.2 (47612), Serivce-ID DS0318 I can't login online anymore. (Hail praise the update pace we all are in, not only we are manged by it - we work for it)

To Reproduce

  1. Log In
  2. Tap on Mehr (Item) , then Online Banking
  3. Try to scan the qr code at banking.c24.de
  4. Nothing happens (Yes, I've tried different monitor settings and scale etc. too)

NOTE: My other qr-code App recognized the qr-code perfectly, and tried to open a web link, but eventually failed because it's the wrong program to open the link.

Expected behavior The 6 digits needed to access the online banking web frontend should appear on screen.

System Android 9 - EMUI 9.1.0

matterharz commented 5 months ago

@DrBarns

Can confirm this too, after recent update to Version 2.73.1.2 (47612), Serivce-ID DS0318 I can't login online anymore. (Hail praise the update pace we all are in, not only we are manged by - it we work for it)

I was about to update to that version, so I just have to ask what exactly do you mean by that? Have you been locked out of the app completely or can't you just not get the web/online banking feature to work?

There have been multiple warnings over the past weeks that my phone wasn't safe due to using a custom ROM, so I already expected them to lock me out permanently any day after a regular update.

DrBarns commented 5 months ago

I was about to update to that version, so I just have to ask what exactly do you mean by that? Have you been locked out of the app completely or can't you just not get the web/online banking feature to work?

It's the latter, aside from web functionality my bank system works. But I think it's not a good idea to lock customers out, on the premises that they are safer using qr code only, eg. leave them no alternative way to login. Web oriented banking is still alive and most of GitHub users (I guess) would prefer to use the browser instead of an isolated app where data transmission between booking tasks become cumbersome.

j4ncp commented 5 months ago

@DrBarns opening the link behind the qr code when scanned with another app should work and bring you to the C24 app. If it does not, make sure you have the setting "app can open supported links" ("App darf unterstützte Links öffnen") enabled in the general app settings for the C24 app.

matterharz commented 5 months ago

@j4ncp Wow, thanks for the hint. For some reason, this is now working for me, too. I didn't even bother to check between my original post and yours. Thanks so much!

Still not working from within the app, though.