[nasa/nos3#352] Update CryptoLib to 1.3.2

[Donnie-Ice]

To test: Send CFS_RADIO TO_ENABLE_OUTPUT, data should start flowing

[Donnie-Ice]

Local build issue currently:

[jlucas9]

Getting this locally for me as well:

-- Building Dynamic App: cryptolib targets=cpu1
CMake Error at /home/jstar/git/nos3/n4/components/cryptolib/src/CMakeLists.txt:134 (install):
  install TARGETS RUNTIME_DEPENDENCIES is not supported when cross-compiling.

[rjbrown2]

Do we need to do something like this:

if(NOT CMAKE_CROSSCOMPILING)
install(TARGETS cryptolib
RUNTIME_DEPENDENCIES
# Other dependencies here
)
else()
install(TARGETS cryptolib)
endif()

[Donnie-Ice]

Update: Currently trying to update TO for TM decryption/framing before release.

[kevincbruce]

Radio Interface appears to work with CryptoLib, so approving!

[jlucas9]

@rjbrown2 / @Donnie-Ice do you all have the changes to the deployment repo for the new script pushed up too?

[jlucas9]

How do I test encrypted commands?

[Donnie-Ice]

@rjbrown2 / @Donnie-Ice do you all have the changes to the deployment repo for the new script pushed up too?

The only thing changed from the deployment repo is the docker image tag, which is pushed to docker cloud.

How do I test encrypted commands?

In the cryptolib window, type tc, then type vcid 2. Commands being sent should now be encrypted.

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Please upload report for BASE (dev@3de9a90). Learn more about missing BASE report.

Additional details and impacted files

@@          Coverage Diff          @@
##             dev    #401   +/-   ##
=====================================
  Coverage       ?   0.00%           
=====================================
  Files          ?       1           
  Lines          ?       2           
  Branches       ?       0           
=====================================
  Hits           ?       0           
  Misses         ?       2           
  Partials       ?       0           

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

[jlucas9]

Working well for me swapping between VCID 1 (default / clear) and VCID 2 (aes-gcm encrypted).

Can we squash this one with the merge?

[dacarter22]

Encrypted messages seems to be working on my end