it-dgc-verificaC19-android icon indicating copy to clipboard operation
it-dgc-verificaC19-android copied to clipboard

Commit 8665458 introduce a violation of GPDP requirements

Open studiofuga opened this issue 3 years ago • 1 comments

Describe the bug

As from GPDP requirements, verifiers should NOT be able to have any information about the type of vaccination or health state of the verified user.

With commit 8665458 a few icons are shown depending from the state of the vaccinations. This is agains the GPDP recommendations.

Expected behaviour

When a partially valid certificate is scanned (that is: 1 dose out of 2 and current time is inside the validity window), GREEN checkmark should be displayed. When the validity window is NOT yet started (i.e. current time is before starting time) RED cross should be displayed.

Current behaviour

When a partially valid certificate is scanned (that is: 1 dose out of 2 and current time is inside the validity window), BLUE checkmark is displayed. When the validity window is NOT yet started (i.e. current time is before starting time) BLUE mark is displayed.

studiofuga avatar Aug 11 '21 16:08 studiofuga

To solve this problem it is possible to introduce a switch in the UI that allows the operator to switch between a "european" check and a "national" check. In this way if the operator sets the operative mode as "national" he would see the certificate valid (green) also in case of a partial vaccination, if the operator sets the operative mode as "european" he would see the same certificate as not valid (with the red cross). In both in case of a certificate not yet valid the result must be a red cross without any information about the future validity of the certificate.

Summarizing: "National": 1 dose(after 2 weeks), 2 doses, test, recovery = valid 1 dose(before 2 weeks) and current not valid certificates = not valid "European": 2 doses, test, recovery = valid 1 dose and current not valid certificates = not valid

This is only a suggestion, but it seems to me the best solution (otherwise we may need to build two twin apps)

Aldo97 avatar Aug 11 '21 18:08 Aldo97

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] avatar Mar 22 '23 14:03 stale[bot]