Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: rammie/rsh
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: 0.0.7
Choose a base ref
...
head repository: rammie/rsh
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: 0.0.8
Choose a head ref
  • 2 commits
  • 8 files changed
  • 2 contributors

Commits on Mar 26, 2026

  1. Reject VariableNames/MemberKeys in validator, add security test coverage

    - Validator now explicitly rejects ${!prefix@} and ${!name[@]} instead of
      silently passing them through to the executor's catch-all
    - Add tests for ANSI-C quoting, brace expansion, indirect expansion,
      here-documents, and here-strings
    - Document --inherit-env and --allow-redirects security implications
    - Extract assert_rejected_with() test helper to reduce boilerplate
    - Pin mise tool versions
    
    Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
    rammie and claude committed Mar 26, 2026
    Configuration menu
    Copy the full SHA
    d629bd1 View commit details
    Browse the repository at this point in the history
  2. Strip LD_PRELOAD/DYLD vars in --inherit-env, add substitution depth g…

    …uard
    
    Security hardening from audit:
    
    - Strip LD_PRELOAD, LD_LIBRARY_PATH, LD_AUDIT, DYLD_INSERT_LIBRARIES,
      DYLD_FRAMEWORK_PATH, DYLD_LIBRARY_PATH even in --inherit-env mode to
      prevent arbitrary code injection via dynamic linker
    - Add substitution depth cap (16) in both validator and executor to
      prevent stack overflow from deeply nested $(...) (DoS)
    - Return error for arithmetic expansion $((...)) instead of silent empty
    - Add test coverage for sed path traversal, input redirects, compound
      redirects, and nested command substitution
    
    Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
    rammie and claude committed Mar 26, 2026
    Configuration menu
    Copy the full SHA
    1e05c1a View commit details
    Browse the repository at this point in the history
Loading