-
Notifications
You must be signed in to change notification settings - Fork 502
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Properly unmount on Ctrl-C #111
Comments
Thanks for the report, and sorry for not responding for so long. I agree that it would be nice to handle better. Unfortunately I probably won't have time to work on this anytime soon, but I'll keep the bug open. |
Was this fixed? I was not able to reproduce this issue:
|
I'm not aware of any deliberate changes. It may have been a side effect of something else... |
Looking at libfuse source code, it seems libfuse handles SIGINT specially and does a graceful exit with umount. Another advice is to use option I think this issue can be closed. |
I don't think libfuse is able to deal with the SSH process being Ctrl-C'ed, this will need treatment in sshfs. |
You are right. Sorry for the noise, I didn't read the description properly. |
@Nikratio if you can point me to where the ssh/sftp (on macOS "ssh" is the process listed) is called, I'll provide the fix to handle when the SSH process quits on a SIGINT. |
@keithmendozasr I did some code excavation and it looks like connect_remote is always called on startup. I think |
At least with regard to macOS, this issue has been addressed in macFUSE 4.6.2. Since 4.6.2 the |
In my local network, I am running the following command (user and remote name anonymised):
When using CTRL+C while entering the password, the directory will be mounted with an unconnected endpoint, so it's there but not usable:
This means that re-trying to mount it will fail:
The workaround is to manually unmount it and then try again:
However, it would be nice if
sshfs
failed gracefully and automatically unmounted the non-connected directory for me.The text was updated successfully, but these errors were encountered: