Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #75 +/- ##
==========================================
- Coverage 94.30% 94.21% -0.10%
==========================================
Files 13 13
Lines 738 882 +144
==========================================
+ Hits 696 831 +135
- Misses 42 51 +9 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Pull request overview
Adds a Linux md5sum fallback when remote system detection fails.
Changes:
- Handles
ChannelOpenErrorandProcessErrorfromuname. - Adds regression coverage for session failure fallback.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
sshfs/spec.py |
Implements checksum fallback logic. |
tests/test_sshfs.py |
Tests fallback to md5sum. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
|
Hi! I wanted to follow up and ask whether this PR is still useful and whether the fallback behaviour matches the project’s expectations. I’m happy to address any requested changes or update the tests if needed. Thanks for taking a look. |
Refs #16.
checksum()currently callsunamebefore trying the checksum command. On servers where opening a shell session forunamefails, this preventsmd5sumfrom being attempted even when it is available.This falls back to the Linux
md5sumpath whenunamecannot be executed, and adds a regression test for that case.Tested with: