Skip to content

RC v1.4.2#491

Merged
Donnie-Ice merged 175 commits into
mainfrom
dev
Sep 22, 2025
Merged

RC v1.4.2#491
Donnie-Ice merged 175 commits into
mainfrom
dev

Conversation

@Donnie-Ice
Copy link
Copy Markdown
Collaborator

@Donnie-Ice Donnie-Ice commented Sep 22, 2025

All Submissions:

  • Have you followed the guidelines in our Contributing document?
  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?

New Feature Submissions:

  • Does your submission pass tests?

Changes to Core Features:

  • Have you added an explanation of what your changes do and why you'd like us to include them?

How do you test these changes?

mirkobitetto and others added 30 commits March 26, 2025 15:01
feat: add fuzzing harness and infrastructure;
…:nasa/CryptoLib into 454-use-ai-to-check-for-ccsds-compliance
… folders due to hard linking being required;
Donnie-Ice and others added 22 commits July 25, 2025 18:27
[nasa/nos3#688] Updates from Donnie-Ice for NOS3 vcid 3 fix;
* Initial plan

* Standardize error returns: Update crypto_config.c and sa_interface functions

Co-authored-by: porfanid <19299306+porfanid@users.noreply.github.com>

* Improve error propagation: Check return values of key_init and mc_initialize

Co-authored-by: porfanid <19299306+porfanid@users.noreply.github.com>
[nasa/nos3#755] fix aos bugs, allow standalone to parse aos
Advisory fix 1
@Donnie-Ice Donnie-Ice requested a review from dccutrig September 22, 2025 15:59
@Donnie-Ice Donnie-Ice self-assigned this Sep 22, 2025
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Sep 22, 2025

Codecov Report

❌ Patch coverage is 58.88502% with 354 lines in your changes missing coverage. Please review.
✅ Project coverage is 81.19%. Comparing base (e56b1fb) to head (3e0ad81).
⚠️ Report is 754 commits behind head on main.

Files with missing lines Patch % Lines
src/core/crypto_config.c 21.73% 132 Missing and 30 partials ⚠️
src/core/crypto_aos.c 40.94% 60 Missing and 15 partials ⚠️
src/core/crypto_tm.c 59.32% 32 Missing and 16 partials ⚠️
src/core/crypto_tc.c 42.42% 25 Missing and 13 partials ⚠️
src/sa/internal/sa_interface_inmemory.template.c 67.21% 15 Missing and 5 partials ⚠️
src/core/crypto.c 44.44% 10 Missing ⚠️
...gcrypt/cryptography_interface_libgcrypt.template.c 66.66% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #491      +/-   ##
==========================================
- Coverage   83.51%   81.19%   -2.33%     
==========================================
  Files          39       39              
  Lines       11074    12235    +1161     
  Branches      835     1053     +218     
==========================================
+ Hits         9249     9934     +685     
- Misses       1518     1905     +387     
- Partials      307      396      +89     

☔ 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.

Copy link
Copy Markdown
Contributor

@dccutrig dccutrig left a comment

Choose a reason for hiding this comment

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

Approved, worth mentioning previously we had tagged dev then merged to main, looks like a couple merges post-tagging were missed - so this main tag is bigger than usual.

@Donnie-Ice Donnie-Ice merged commit 309c1c4 into main Sep 22, 2025
18 checks passed
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.

9 participants