Skip to content

fix: RC credentials env vars + dry-run support (S510, FM-11)#14

Merged
ozymandiashh merged 1 commit intomainfrom
fix/final-cleanup
Mar 22, 2026
Merged

fix: RC credentials env vars + dry-run support (S510, FM-11)#14
ozymandiashh merged 1 commit intomainfrom
fix/final-cleanup

Conversation

@ozymandiashh
Copy link
Copy Markdown
Owner

Summary

  • S510 (Low): RC API credentials no longer visible in ps aux - passed via RCLONE_RC_USER/RCLONE_RC_PASS environment variables
  • FM-11 (Critical): Server now handles dry_run parameter from all start endpoints, passes --dry-run to rclone

Test plan

  • ps aux during transfer shows no --rc-user/--rc-pass
  • MCP start_transfer with dry_run=True does not transfer real files
  • python3 -m pytest -q all pass (501 passed)
  • python3 -m ruff check cloudhop/ clean

…-11)

- S510: Pass --rc-user/--rc-pass via RCLONE_RC_USER/RCLONE_RC_PASS
  environment variables instead of command line arguments
- FM-11: Read dry_run from request body and pass --dry-run to rclone
@ozymandiashh ozymandiashh merged commit b26feb3 into main Mar 22, 2026
4 of 15 checks passed
@ozymandiashh ozymandiashh deleted the fix/final-cleanup branch March 22, 2026 12:12
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