Skip to content

RC v1.4.0 #448

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

Merged
merged 190 commits into from
Apr 25, 2025
Merged

RC v1.4.0 #448

merged 190 commits into from
Apr 25, 2025

Conversation

Donnie-Ice
Copy link
Contributor

@Donnie-Ice Donnie-Ice commented Apr 3, 2025

Donnie-Ice and others added 30 commits June 13, 2024 14:36
… deactivating a bunch of keys. Tests are being worked pre-development
…xt to increment_iv_arsn

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
…plaintext

[#356] Renamed do_encrypt_plaintext to encrypt, do_encr…

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
[#47] Reworked setARSN to match SDLS-EP 3.3.3.7.3.4 exe…

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
@Donnie-Ice Donnie-Ice self-assigned this Apr 3, 2025
@codecov-commenter
Copy link

codecov-commenter commented Apr 3, 2025

Codecov Report

Attention: Patch coverage is 82.55408% with 250 lines in your changes missing coverage. Please review.

Project coverage is 74.99%. Comparing base (e56b1fb) to head (d72b9e2).
Report is 301 commits behind head on main.

Files with missing lines Patch % Lines
src/sa/internal/sa_interface_inmemory.template.c 31.09% 79 Missing and 3 partials ⚠️
src/core/crypto_tm.c 59.75% 23 Missing and 10 partials ⚠️
src/core/crypto_tc.c 53.73% 18 Missing and 13 partials ⚠️
test/unit/ut_ep_key_mgmt.c 87.87% 24 Missing ⚠️
src/core/crypto_aos.c 69.23% 12 Missing and 8 partials ⚠️
src/core/crypto_key_mgmt.c 76.27% 12 Missing and 2 partials ⚠️
test/unit/ut_ep_sa_mgmt.c 86.13% 14 Missing ⚠️
src/core/crypto.c 84.88% 8 Missing and 5 partials ⚠️
src/core/crypto_config.c 30.76% 8 Missing and 1 partial ⚠️
test/unit/ut_ep_mc.c 84.61% 6 Missing ⚠️
... and 2 more
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #448      +/-   ##
==========================================
- Coverage   83.51%   74.99%   -8.53%     
==========================================
  Files          39       80      +41     
  Lines       11074    23341   +12267     
  Branches      835     1872    +1037     
==========================================
+ Hits         9249    17505    +8256     
- Misses       1518     5168    +3650     
- Partials      307      668     +361     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@Donnie-Ice Donnie-Ice linked an issue Apr 3, 2025 that may be closed by this pull request
@dccutrig
Copy link
Contributor

Confirming this is the commit that JPL has been testing with? @Donnie-Ice

@Donnie-Ice
Copy link
Contributor Author

Confirming this is the commit that JPL has been testing with? @Donnie-Ice

They were testing from dev toward the end, so yes

@Donnie-Ice Donnie-Ice merged commit 0d58c9a into main Apr 25, 2025
17 checks passed
@@ -67,34 +67,6 @@ void Crypto_tmPrint(TM_t *tm_frame)
printf("Current TM in memory is: \n");
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is going on in this print statement. Should we just remove it entirely at this point?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

RC 1.4.0
5 participants