Skip to content

improve sftp mounts#2105

Open
alexanderadam wants to merge 3 commits intoGSConnect:mainfrom
alexanderadam:fix/improve_sftp_mounts
Open

improve sftp mounts#2105
alexanderadam wants to merge 3 commits intoGSConnect:mainfrom
alexanderadam:fix/improve_sftp_mounts

Conversation

@alexanderadam
Copy link
Copy Markdown

@alexanderadam alexanderadam commented Dec 3, 2025

Well, this stuff is super complicated.
I probably did a lot of things not right but it works for me pretty well now.

I removed the symlink logic: I love it visually but it is indeed suuuuuuuuuuuper slow for some reason.
And I'm not the only one who realised this.

And regarding the deep link mount: I think it is related to this upstream bug.
So we can't tackle this until the SFTP issue is solved but this PR will simply open Nautilus at the mount point for now.

…ooor we decide to abandon gvfs. 😉

fixes #1203 #1861 #1857 #1794 #1610

@daniellandau
Copy link
Copy Markdown
Member

@ferdnyc you looked into this in the past, could you maybe try to review this?

Copy link
Copy Markdown
Member

@daniellandau daniellandau left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I haven't really used the mount functionality and reading up on the discussions it seems to have a long history of issues. Testing how it worked before this PR and after, this seems to be an improvement and doesn't seem to have direct downsides, except probably breaking/code conflicting other proposed solutions such as #1921 and #1793.

I'm inclined to go with this, let's say in one week from now if @ferdnyc or @andyholmes don't come with differing opinions.

I am aware that the nautilus sidebar continues to have a broken "permission denied" item after this PR too, but it has that now as well so it's not making things worse.

@andyholmes
Copy link
Copy Markdown
Collaborator

The sftp plugin has always been a bit flaky, so the most important thing is the code has been tested, but it looks reasonable to me. Outside of that, I think there might be some users who still depend on the symlink for a reliable path; not sure though.

@daniellandau
Copy link
Copy Markdown
Member

After running this now for a while, it seems to leak umnount actions

image

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.

SFTP: Device does not mount

3 participants