Skip to content

Quote bitcoin kubectl command arguments#808

Draft
Asylian21 wants to merge 1 commit into
bitcoin-dev-project:mainfrom
Asylian21:codex/quote-bitcoin-kubectl-commands
Draft

Quote bitcoin kubectl command arguments#808
Asylian21 wants to merge 1 commit into
bitcoin-dev-project:mainfrom
Asylian21:codex/quote-bitcoin-kubectl-commands

Conversation

@Asylian21
Copy link
Copy Markdown

Summary

  • build bitcoin RPC and log kubectl commands with shlex.join so dynamic arguments are shell-quoted consistently
  • avoid shell=True when reading captured message files
  • add a regression check that shell metacharacters remain single command arguments

Validation

  • PYTHONPATH=.:src:resources/scenarios:test uv run --frozen python test/bitcoin_rpc_args_test.py
  • uvx ruff check src/warnet/bitcoin.py test/bitcoin_rpc_args_test.py
  • git diff --check

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