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.

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

release(v0.1.1): exposed data and bug fix patch #3

Merged
merged 6 commits into from
Dec 1, 2023
Merged

Conversation

andrea-deluca
Copy link
Owner

This patch resolves some problems detected during demo development and testing. In particular,
a bug has been found during Subject Public Key parsing: the first public key byte was removed and therefore
parsed key was invalid. Moreover, a bug that was causing a fatal error during PassiveAuthentication has been fixed.

In addition, more info are now exported from the NFCPassportModel, such as security protocols,
data groups available and read, hashes and X509 certificate details stored within the SOD.

andrea-deluca and others added 6 commits November 28, 2023 12:46
- Properties strored into MRZ struct are now exposed
- Fix some string format in PersonalDetails model
- dates are now formatted
- filler chars removed
- X509CertificateDetails struct is now exported
- NFCPassportModel now export more info about security protocols, data groups and certificate
Fixed a bug that was causing a fatal error during PassiveAuthentication:
- digest alg is now nullable in SignedData
- if digest alg is nil then use try to use SHA1

Fixed an issue during the SubjectPublicKeyInfo parsing
fix(crypto): pubkey, signed data and PAHandler
@andrea-deluca andrea-deluca added the enhancement New feature or request label Dec 1, 2023
@andrea-deluca andrea-deluca self-assigned this Dec 1, 2023
@andrea-deluca andrea-deluca merged commit be3a9f7 into main Dec 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant