search

corona-warn-app / cwa-wishlist

Central repository to collect community feature requests and improvements. The CWA development ends on May 31, 2023. You still can warn other users until April 30, 2023. More information:
https://coronawarn.app/en/faq/#ramp_down
Apache License 2.0
105 stars 14 forks source link

Detect lower saxony QR codes and warn user #545

Closed Manawyrm closed 3 years ago

Manawyrm commented 3 years ago

Current Implementation

When erroneously scanning a QR code listed on the paperwork of the vaccination centers in lower saxony (for use as a vaccination certificate) the user will get a confusing "VC_PREFIX_INVALID" message.

Suggested Enhancement

In lower saxony, regular QR codes with 8 character A-Z, 0-9 are present on the letters from the vaccination centers. The regex ^[0-9A-Z]{8}$ matches against these NDS QR codes and could maybe used to detect them and warn the user?

I've seen some confusion in family members, because they tried to register with that code.

My proposal for such an error message would be something like:

Der gescannte QR-Code gehört zu Ihrer Anmeldung beim Impfzentrum und ist kein gültiges Impfzertifikat. Sie erhalten Ihr Impfzertifikat per Post oder von einer Apotheke.

Expected Benefits

Users wouldn't be confused by confusing errors and the clear error message would tell them what they need to do.

Manawyrm commented 3 years ago

For completeness, here's a screenshot:

And here are 2 (fake) NDS QR codes: 3DZJOU8N:

ABCD1234:

MikeMcC399 commented 3 years ago
Ein-Tim commented 3 years ago

The same goes for the QR-Codes on the paper work of the Bavarian vaccination centers (not sure about the regex though).

In General I think the error message should be improved (maybe with a link to the FAQ entry which explains this - one will hopefully be published soon https://github.com/corona-warn-app/cwa-website/issues/1343)

Ein-Tim commented 3 years ago

We will get a hot fix for improved error handling reg. this: https://github.com/corona-warn-app/cwa-app-android/pull/3400

The error message then will include a link to the FAQs!

Manawyrm commented 3 years ago

Here's a report about the same problem in Munich with a dump of the data in Munich. It seems to be a JSON-blob with an ASCII prefix?

https://twitter.com/HillierHillier/status/1402660888843325445

Ein-Tim commented 3 years ago

@Manawyrm Yeah, there's nothing we can do about this better than the hot fix release 2.3.3 which will add a link to a FAQ entry.

MikeMcC399 commented 3 years ago

Note the new FAQ entry

heinezen commented 3 years ago

We have improved the error message in CWA version 2.3.3, so scanning the wrong QR code is less confusing. With the automatic distribution of vaccine certificates with every vaccination this will hopefully be even lesser of a problem.

Corona-Warn-App Open Source Team