Skip to content

Python client/test app bug fixes#75

Open
jppan-jpl wants to merge 2 commits intodevfrom
python-bug-fixes
Open

Python client/test app bug fixes#75
jppan-jpl wants to merge 2 commits intodevfrom
python-bug-fixes

Conversation

@jppan-jpl
Copy link
Copy Markdown
Contributor

  • Addressed bugs re: flag handling in test app
  • Removed bitstring from python client
  • refactored to use bitshift logic or struct bytearray extend in both the python client and test app
  • cleaned up default test app properties file
  • hit at least one of the python files with the formatter stick (sorry!)

… output. fixing bugs in KmcSdlsClient.py related to hex output. Modifying default managed parameters for aos/tm in kmc_sdls_test_app.properties.
…rwise exit 0. changed aos to not check IV, was getting errors with local db (unsure if it's a bug)
cryptolib.aos.create_ecf=true
cryptolib.aos.check_fecf=false
cryptolib.aos.ignore_antireplay=false
cryptolib.aos.ignore_antireplay=true
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

i'm not sure if this is necessary; i was getting incrementing IV errors in my testing, could be an SADB problem.

Copy link
Copy Markdown
Contributor Author

@jppan-jpl jppan-jpl Apr 21, 2026

Choose a reason for hiding this comment

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

most changes here are formatting, sorry!

modified the Frame/TC/TM/AOS classes to use bitshift packing instead of the brittle bit "string" and hex mishmash. changes were tested before/after to ensure correctness.

also modified this app to only return an error code (non-zero exit) if an error occurs.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

changed all default frame lengths to 128 bytes, added a few more configs for TM/AOS.

Copy link
Copy Markdown
Contributor Author

@jppan-jpl jppan-jpl Apr 21, 2026

Choose a reason for hiding this comment

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

removed bitstring dependency, and refactored to bitshift and struct for header hex conversions. tested with before/after outputs.

@@ -1,3 +1,2 @@
cffi==1.14.6
invoke==1.6.0
bitstring==4.3.1 No newline at end of file
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

removed bitstring from requirements

@sonarqubecloud
Copy link
Copy Markdown

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.

1 participant