Skip to content

Expand curl support with comprehensive coverage#3

Merged
ldayton merged 1 commit intomainfrom
add-curl-support
Jan 8, 2026
Merged

Expand curl support with comprehensive coverage#3
ldayton merged 1 commit intomainfrom
add-curl-support

Conversation

@ldayton
Copy link
Copy Markdown
Owner

@ldayton ldayton commented Jan 8, 2026

Summary

  • Add 199 tests covering curl CLI commands in dedicated test_curl.py
  • Add unsafe patterns: --json, -K/--config, --ftp-create-dirs, --mail-from/--mail-rcpt
  • Add FTP quote command parsing to allow safe commands (PWD, LIST, etc.) and block unsafe ones (DELE, MKD, etc.)
  • Fix --request= form to properly allow OPTIONS/TRACE methods

Test plan

  • All new tests pass (199 tests)
  • Linter passes
  • Existing tests still pass (3394 total)

Add 199 tests covering curl CLI commands in dedicated test_curl.py.
Handle new unsafe patterns: --json, --config, --ftp-create-dirs,
--mail-from/--mail-rcpt, and FTP quote commands. Fix --request=
form to properly allow OPTIONS/TRACE methods.
@ldayton ldayton merged commit 57e1b05 into main Jan 8, 2026
@ldayton ldayton deleted the add-curl-support branch January 8, 2026 06:42
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