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.

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

Encryption - ARK #24489

Open
wants to merge 8 commits into
base: main
Choose a base branch
from
Open

Encryption - ARK #24489

wants to merge 8 commits into from

Conversation

patrikpordi
Copy link
Contributor

@patrikpordi patrikpordi commented Mar 11, 2025

Solved Problem

Making creating encrypted board files and then log download and decryption easier

Solution

Creating scripts to generate keys, dowload the logs and then decrypt them

Context

Navigate to the Tools folder
python3 key.py

make your ark board and upload it to the flight controller
python3 download_logs.py serial:/dev/ttyACM1:57600 (I have only tested the serial)
python3 decrypt_logs.py
Your logs should be found in
PX4-Autopilot/log/decrypted_logs$

@patrikpordi
Copy link
Contributor Author

@dakejahl @AlexKlimaj

Copy link
Contributor

@dakejahl dakejahl left a comment

Choose a reason for hiding this comment

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

Let's put these new scripts in their own dir, maybe Tools/log_encryption/. Let's also add a README to this new dir that explains how to use

Copy link
Contributor

@hamishwillee hamishwillee left a comment

Choose a reason for hiding this comment

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

There is nothing about these scripts that says they are not generally useful. So when this merges, can you update https://docs.px4.io/main/en/dev_log/log_encryption.html ?

@dakejahl
Copy link
Contributor

Rather than create a new file with a new name decrypt_logs.py why don't we update the existing decrypt_ulog.py?

Adjusted the log downloader to handle multiple entry responses from the FC
Edited README
## Prerequisites

- Python 3.x
- pip3 install --user pycryptodome pymavlink
Copy link
Contributor

Choose a reason for hiding this comment

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

Can you add this to the Tools/setup/requirements.txt file?

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.

5 participants