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

CryptoLib#140 - TM Authentication #155

Merged
merged 53 commits into from
Apr 14, 2023
Merged

CryptoLib#140 - TM Authentication #155

merged 53 commits into from
Apr 14, 2023

Conversation

jlucas9
Copy link
Collaborator

@jlucas9 jlucas9 commented Apr 13, 2023

No description provided.

rjbrown2 and others added 30 commits November 9, 2022 12:32
@rjbrown2
Merge pull request #135 from nasa/dev
72f4ff5 
Dev Merge into Main
@dccutrig
Add TM info to config structs
feab082
@dccutrig
Add SA12 for testing, update some print statements
1cc43e2
@dccutrig
WIP on TM UT tests
50e6493
@dccutrig
Progress checkin
707a037
@dccutrig
Condense FECF enums to one enum to handle different cases
399605a
@dccutrig
WIP - TM_frame to static, genericize add_gvcid_param calls, add worki…
7a8ac96 
…ng TM_Apply UT'
@dccutrig
UT changes to support generic managed paramater changes
87b8eaf
@dccutrig
Genericize managed parameter max_frame_size
b25c1f8
@dccutrig
Modify TM print debugs
2a6a1a4
@dccutrig
Fix rogue rind and replace breaking UT
7bd64bc
@dccutrig
Separate TC/TM UT setup, modify config enums, WIP on tm_Apply, all UT…
495ffba 
…s pass
@dccutrig
Add UT to check allow non-replacement of FECF
3d52542
@dccutrig
WIP on TM_Apply Authentication
baeea44
@dccutrig
Enable more TM_Apply UTs
c146a40
@dccutrig
Fix typos in comment of encryption key
ad2b5fe
@dccutrig
Updating a python verification test for CMAC use with TM AES test
97cd0b7
@dccutrig
WIP on Tm_Apply, authentication for CMAC, add TM_Prepare_AAD function
617b65d
@dccutrig
UT for TM_Apply CMAC
9c9cc3c
@dccutrig
Add UT for Tm_Apply CMAC w/ bitmask of zeros
9bbc99d
@dccutrig
Add UTs TM_Apply SHA 256 HMAC
720da28
@dccutrig
Add UTs TM_Apply HMAC SHA512
f7640df
@dccutrig
Update crypto_print for TM secondary hdr updates
eb8d021
@dccutrig
Update Secondary Header smarts to update index as necessary
55fedb6
@dccutrig
TM_Apply UT free truth data
879ccdf
@dccutrig
Update UT comments, WIP on initial UT for secondary header
a2863a7
@dccutrig
Add UT for TM_Apply secondary header presence
4d1c123
@dccutrig
Minor cleanup
6e56e5b
@dccutrig
Minor cleanup
80f5254
@dccutrig
Add TM_Apply UT for Secondary headers w/ MAC
8a2c969
@jlucas9 jlucas9 self-assigned this Apr 13, 2023
@dccutrig
Copy link
Contributor

Just wanted to leave a comment that this is functional - and merging back into dev for other work. This does need cleaned up, and will be cleaned up better in the encryption branch that is upcoming.

@rjbrown2
Copy link
Contributor

Should we make sure all of these tests are passing before we merge into?

@jlucas9
Copy link
Collaborator Author

jlucas9 commented Apr 14, 2023

Note that #156 was created due to the issues above.

@jlucas9 jlucas9 merged commit a5467c4 into dev Apr 14, 2023
9 checks passed
@jlucas9 jlucas9 deleted the 140-tm-support branch April 28, 2023 12:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dccutrig dccutrig dccutrig approved these changes

@rjbrown2 rjbrown2 Awaiting requested review from rjbrown2

Assignees

@jlucas9 jlucas9

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@jlucas9 @dccutrig @rjbrown2