BlockchainCommons / GordianQRTool-iOS

UR & QR code storage for iOS and MacOS
Other
5 stars 7 forks source link

Displayed QR codes are not case-preserving for URs. #45

Closed wolfmcnally closed 3 years ago

wolfmcnally commented 3 years ago

This QR code contains UR:CRYPTO-SSKR/GODWMDBGCPAEHYOLJEGEFPSNCWADWDGOLEVEOXMWVEKTEOKGOXCF. This is properly encoded in upper case, to take advantage of the QR code alphanumeric encoding mode.

image

The QR code retrieved from QRVault contains ur:crypto-sskr/godwmdbgcpaehyoljegefpsncwadwdgoleveoxmwvekteokgoxcf. This is the same UR, but transformed to lower case. Since URs are not case sensitive, this only results in somewhat less-efficient encoding in the actual optical code.

image

I tried storing a QR code simply containing the text "UPPER-CASE lower-case" in QRVault, and it successfully made the round trip preserving case, so I suspect this issue is specific to stored URs.