Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[satosa] QR code error correction level #285

Open
Zicchio opened this issue Oct 23, 2024 · 0 comments
Open

[satosa] QR code error correction level #285

Zicchio opened this issue Oct 23, 2024 · 0 comments
Assignees
@peppelinux
Labels
help wanted Need help to resolve this
Milestone
0.9.1

Comments

@Zicchio
Copy link
Collaborator

Zicchio commented Oct 23, 2024

Accorrding to italian specs, the QR code where request data can be downloaded must be set at correction level Q.
See
https://github.com/italia/eudi-wallet-it-docs/blob/fdd9cf44e69bd6eeb4cfb11684dd6f5e0e7c6a57/docs/en/remote-flow.rst?plain=1#L196

It is not clear to me if the current QR code satify this requirement.

If I'm not mistaken, the current implementatio uses this library
https://github.com/bitjson/qr-code
which (maybe??) create QR codes by default at level H and is not configurable
https://github.com/bitjson/qr-code/blob/de60f23e80d5c7477a1d58512e32eb6751b85f88/src/components/qr-code/qr-code.tsx#L154-L158
but I'm not sure with this claim as how the library operates is unclear to me.
@Zicchio Zicchio added the help wanted Need help to resolve this label Oct 23, 2024
@peppelinux peppelinux added this to the 0.9.1 milestone Oct 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@peppelinux peppelinux

Labels
help wanted Need help to resolve this
Projects
EUDI WALLET IT Python
Status: No status
Milestone
0.9.1
Development

No branches or pull requests
2 participants
@peppelinux @Zicchio