Update dependencies

This change largely updates the dependencies and adds on to the
dependabot config to have it manage dependency updates in the future.

I also did a few minor tweaks that don't really affect users much.

Author: JacobCallahan

.github/dependabot.yml

@@ -5,6 +5,13 @@
 version: 2
 updates:
 
+  # Maintain Rust Cargo dependencies
+  - package-ecosystem: "cargo"
+    directory: "/"
+    schedule:
+      interval: "daily"
+    labels:
+      - "dependencies"
   # Maintain dependencies for our GitHub Actions
   - package-ecosystem: "github-actions"
     directory: "/"

.github/workflows/build_and_test.yml

@@ -81,7 +81,7 @@ jobs:
         with:
           target: ${{ matrix.platform.target }}
           # args: --release --out dist --find-interpreter  <-- TODO: Add this back when maturin supports it
-          args: --release --out dist -i 3.8 3.9 3.10 3.11 3.12 3.13 pypy3.8 pypy3.9 pypy3.10
+          args: --release --out dist -i 3.8 3.9 3.10 3.11 3.12 3.13 pypy3.9 pypy3.10
           # sccache: 'true'
           manylinux: auto
           before-script-linux: |
@@ -137,7 +137,7 @@ jobs:
           PKG_CONFIG_PATH: "/usr/share/miniconda/lib/pkgconfig/"
         with:
           target: ${{ matrix.platform.target }}
-          args: --release --out dist -i 3.8 3.9 3.10 3.11 3.12 3.13 pypy3.8 pypy3.9 pypy3.10
+          args: --release --out dist -i 3.8 3.9 3.10 3.11 3.12 3.13 pypy3.9 pypy3.10
           # sccache: 'true'
           manylinux: musllinux_1_2
           before-script-linux: |
@@ -224,7 +224,7 @@ jobs:
         uses: PyO3/maturin-action@main
         with:
           target: ${{ matrix.target }}
-          args: --release --out dist -i 3.8 3.9 3.10 3.11 3.12 3.13 pypy3.8 pypy3.9 pypy3.10
+          args: --release --out dist -i 3.8 3.9 3.10 3.11 3.12 3.13 pypy3.9 pypy3.10
           sccache: 'true'
       - name: Upload wheels
         uses: actions/upload-artifact@v4

.github/workflows/release.yml

@@ -72,7 +72,7 @@ jobs:
           OPENSSL_NO_VENDOR: 1
         with:
           target: ${{ matrix.platform.target }}
-          args: --release --out dist -i 3.8 3.9 3.10 3.11 3.12 3.13 pypy3.8 pypy3.9 pypy3.10
+          args: --release --out dist -i 3.8 3.9 3.10 3.11 3.12 3.13 pypy3.9 pypy3.10
           # sccache: 'true'
           manylinux: auto
           before-script-linux: |
@@ -121,7 +121,7 @@ jobs:
           PKG_CONFIG_PATH: "/usr/share/miniconda/lib/pkgconfig/"
         with:
           target: ${{ matrix.platform.target }}
-          args: --release --out dist -i 3.8 3.9 3.10 3.11 3.12 3.13 pypy3.8 pypy3.9 pypy3.10
+          args: --release --out dist -i 3.8 3.9 3.10 3.11 3.12 3.13 pypy3.9 pypy3.10
           # sccache: 'true'
           manylinux: musllinux_1_2
           before-script-linux: |
@@ -200,7 +200,7 @@ jobs:
         uses: PyO3/maturin-action@main
         with:
           target: ${{ matrix.target }}
-          args: --release --out dist -i 3.8 3.9 3.10 3.11 3.12 3.13 pypy3.8 pypy3.9 pypy3.10
+          args: --release --out dist -i 3.8 3.9 3.10 3.11 3.12 3.13 pypy3.9 pypy3.10
           sccache: 'true'
       - name: Upload wheels
         uses: actions/upload-artifact@v4

.github/workflows/weekly_build_and_test.yml

@@ -78,7 +78,7 @@ jobs:
           OPENSSL_NO_VENDOR: 1
         with:
           target: ${{ matrix.platform.target }}
-          args: --release --out dist -i 3.8 3.9 3.10 3.11 3.12 3.13 pypy3.8 pypy3.9 pypy3.10
+          args: --release --out dist -i 3.8 3.9 3.10 3.11 3.12 3.13 pypy3.9 pypy3.10
           # sccache: 'true'
           manylinux: auto
           before-script-linux: |
@@ -134,7 +134,7 @@ jobs:
           PKG_CONFIG_PATH: "/usr/share/miniconda/lib/pkgconfig/"
         with:
           target: ${{ matrix.platform.target }}
-          args: --release --out dist -i 3.8 3.9 3.10 3.11 3.12 3.13 pypy3.8 pypy3.9 pypy3.10
+          args: --release --out dist -i 3.8 3.9 3.10 3.11 3.12 3.13 pypy3.9 pypy3.10
           # sccache: 'true'
           manylinux: musllinux_1_2
           before-script-linux: |
@@ -221,7 +221,7 @@ jobs:
         uses: PyO3/maturin-action@main
         with:
           target: ${{ matrix.target }}
-          args: --release --out dist -i 3.8 3.9 3.10 3.11 3.12 3.13 pypy3.8 pypy3.9 pypy3.10
+          args: --release --out dist -i 3.8 3.9 3.10 3.11 3.12 3.13 pypy3.9 pypy3.10
           sccache: 'true'
       - name: Upload wheels
         uses: actions/upload-artifact@v4

Cargo.toml

@@ -9,9 +9,9 @@ name = "hussh"
 crate-type = ["cdylib"]
 
 [dependencies]
-openssl = { version = "0.10", features = ["vendored"] }
-pyo3 = "0.22"
-shellexpand = "2.1"
+openssl = { version = "0.10.68", features = ["vendored"] }
+pyo3 = "0.23.1"
+shellexpand = "3.1.0"
 # ssh2 = "0.9"
 # temporary until ssh2#312 makes it into a release. probably 0.9.5
 ssh2 = { git = "https://github.com/alexcrichton/ssh2-rs", branch = "master" }

src/connection.rs

@@ -117,7 +117,7 @@ impl SSHResult {
     // The __str__ method for the SSHResult class
     fn __str__(&self) -> PyResult<String> {
         Ok(format!(
-            "stdout:\n{}stderr:\n{}status: {}",
+            "stdout:\n{}\nstderr:\n{}\nstatus: {}",
             self.stdout, self.stderr, self.status
         ))
     }

src/lib.rs

@@ -10,9 +10,6 @@ fn hussh(_py: Python, m: &Bound<'_, PyModule>) -> PyResult<()> {
     m.add_class::<connection::SSHResult>()?;
     m.add_class::<connection::InteractiveShell>()?;
     m.add_class::<connection::FileTailer>()?;
-    m.add(
-        "AuthenticationError",
-        _py.get_type_bound::<AuthenticationError>(),
-    )?;
+    m.add("AuthenticationError", _py.get_type::<AuthenticationError>())?;
     Ok(())
 }

tests/conftest.py

@@ -30,17 +30,20 @@ def run_test_server(ensure_test_server_image):
     client = docker.from_env()
     try:  # check to see if the container is already running
         container = client.containers.get("hussh-test-server")
+        managed = False
     except docker.errors.NotFound:  # if not, start it
         container = client.containers.run(
             TEST_SERVER_IMAGE,
             detach=True,
             ports={"22/tcp": 8022},
             name="hussh-test-server",
         )
+        managed = True
         time.sleep(5)  # give the server time to start
     yield container
-    container.stop()
-    container.remove()
+    if managed:
+        container.stop()
+        container.remove()
     client.close()
 
 

tests/test_connection.py

@@ -180,6 +180,14 @@ def test_pty_shell_context(conn):
     assert sh.result.status != 0
 
 
+@pytest.mark.skip("not yet implemented")
+def test_hangup_shell_context(conn):
+    """Test that we can hang up a running shell while a previous command is still running."""
+    with conn.shell() as sh:
+        sh.send("tail -f /dev/random")
+    assert sh.result.stdout
+
+
 def test_session_timeout():
     """Test that we can trigger a timeout on session handshake."""
     with pytest.raises(TimeoutError):