-
Notifications
You must be signed in to change notification settings - Fork 4
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
feat: write atkeys to file #428
base: trunk
Are you sure you want to change the base?
Conversation
95939bc
to
c8d5081
Compare
c8d5081
to
d80994f
Compare
@JeremyTubongbanua apkamSymmetricKey is now a part of the atKeys file, since implementation of APKAM. Could you please add impl to write that into the keys file too. Support for APKAMSymmetricKey is still not integrated into the at_c client. It's currently in this PR #412, could you please cherry pick these commits into this branch. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See earlier comment reg APKAMSymmetricKey
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Address the tests
closes #415
- What I did
3 new functions
atclient_atkeys_file_write_to_path
atclient_atkeys_write_to_atkeys_file
atclient_atkeys_write_to_path
(this function depends on the two functions stated above)test_atkeys_write.c
-> this reads an atkeys file (string version), writes it to a file, then we read the file that we wrote, then use the new atkeys1 struct to pkam authenticate, to ensure that reading was successful- How to verify it
Manual test: read an atkeys file, write it to a second file, then try utilizing the atkeys that was written to the second file
Output
- Description for the changelog
feat: write atkeys to file