Skip to content

Enhance privacy document with NFC data processing details - #81

Open
marinaado wants to merge 3 commits into
mainfrom
marinaado-patch-1
Open

marinaado wants to merge 3 commits into
mainfrom
marinaado-patch-1

Conversation

@marinaado

Copy link
Copy Markdown

Updated the privacy document to include additional data processing steps involving NFC and OCR for personal data extraction.

Added few steps in the purpose, and they probably need to be reflected in the flow diagram below?

Updated the privacy document to include additional data processing steps involving NFC and OCR for personal data extraction.

Added few steps in the purpose, and they probably need to be reflected in the flow diagram below?
@marinaado
marinaado requested a review from Didr October 12, 2026 09:33
@codecov

codecov Bot commented Oct 12, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@Didr Didr left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Several parts of the document are outdated so we can leave the flow diagram update for a separate PR.

Comment thread PRIVACY.md Outdated
Comment thread PRIVACY.md
1. Receive a biometric liveness scan.
2. Take a photo of the photo-ID, perform the OCR read of the data including the MRZ information.
3. Access the photo-ID chip using thE NFC and read the data stored in the document that includes the users personal data and the documents signing certificates and security components stored on the chip (hashes, signatures and keys).
4. Perform active photo-ID authentication (check that the document was not cloned). (Performing the passive authentication of the photo-ID document is not in scope of facetec-api and is left to customers to implement their own flow).

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
4. Perform active photo-ID authentication (check that the document was not cloned). (Performing the passive authentication of the photo-ID document is not in scope of facetec-api and is left to customers to implement their own flow).
4. Perform active and passive photo-ID authentication.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One part of passive authentication is not part of facetec api and this is to build an authoritative source of countries CSCA. If you agree with this, i would rephrase this sentence.

Comment thread PRIVACY.md
2. Take a photo of the photo-ID, perform the OCR read of the data including the MRZ information.
3. Access the photo-ID chip using thE NFC and read the data stored in the document that includes the users personal data and the documents signing certificates and security components stored on the chip (hashes, signatures and keys).
4. Perform active photo-ID authentication (check that the document was not cloned). (Performing the passive authentication of the photo-ID document is not in scope of facetec-api and is left to customers to implement their own flow).
5. Verify that the data from the NFC chip was not tampered with.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is the same as active authentication defined in point 4. We can skip this.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You are half right :) It is the same as the passive authentication so we can skip this indeed, but i would also describe it under that point so that we are clear.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As per my previous comment, facetec api is not performing the passive authentication in its entirety.

Comment thread PRIVACY.md Outdated
marinaado and others added 2 commits October 12, 2026 21:50
Co-authored-by: Didr <Didr@users.noreply.github.com>
Co-authored-by: Didr <Didr@users.noreply.github.com>
@sonarqubecloud

Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants