chore: add TODOs

This commit is contained in:
Benjamin Lipp
2025-07-22 17:42:13 +02:00
parent 9327c2c4f3
commit a5ae83e726
3 changed files with 16 additions and 2 deletions

View File

@@ -5,4 +5,4 @@
* call the old script from the Rosenpass repository's root directory with `./analyze.sh`
* call the new script:
* `nix run .# -- analyze analyze $repo` where `$repo` is the absolute(?) path to the root directory of the Rosenpass repository.
* `nix run .# -- analyze $repo` where `$repo` is the absolute(?) path to the root directory of the Rosenpass repository.

View File

@@ -23,6 +23,18 @@
## Next Steps
* rebase from main
* see if we still need the `extra_args is None` check in `_run_proverif`
* set colors differently to prevent injection attack
* by calling a function
* by prepared statements
* standalone function parse_result_line is no longer necessary
* is the clean function still necessary?
* implement better main function for click
* return an exit status that is meaningful for CI
* rewrite marzipan.awk into Python/LARK
* rewrite cpp into Python/LARK
* integrate the Nix flake into the main Nix flake
* think about next steps
* integrate this upstream, into the CI?
* “make it beautiful” steps? more resiliency to working directory?
@@ -31,3 +43,6 @@
* and rewrite the AST within Python
* reconstruct ProVerif input file for ProVerif
* rewrite our CPP usages into Python/…?
“it replaces the Bash script and is idiomatic Python code”

View File

@@ -111,7 +111,6 @@ def _awk(cpp_prep, awk_prep):
file.write("\nprocess main")
def pretty_output_line(prefix, mark, color, text):
# Vulnerable to injection attacks. Use better API. -- karo
prefix = f"[grey42]{prefix}[/grey42]"