Skip to content
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

Unable to change directories when exposed via Mina SFTP #146

Closed
1 task done
ptirador opened this issue Dec 17, 2020 · 3 comments · Fixed by #148
Closed
1 task done

Unable to change directories when exposed via Mina SFTP #146

ptirador opened this issue Dec 17, 2020 · 3 comments · Fixed by #148
Assignees
Milestone

Comments

@ptirador
Copy link
Contributor

ptirador commented Dec 17, 2020

Task Description

When using S3FS as a file system for Mina SFTP, it's not possible to change the directory (cd command):

sftp> ls -l
d---------   1 OWNER@   GROUP@          0 Jan 25 10:51 bar
d---------   1 OWNER@   GROUP@          0 Jan 25 10:42 foo
sftp> cd foo
Can't change directory: Can't check target

POSIX permissions are not being returned for these directories, which is causing an exception in the SFTP client when attempting to cd to the directory.

Tasks

The following tasks will need to be carried out:

  • When getting S3 Posix File Attributes in S3Utils, if the path is a directory, obtain the bucket ACL, then obtain its ACL grant permissions, convert them to Posix file permissions and return them with the response.

Useful Links

Help

@ptirador
Copy link
Contributor Author

HI @thomascollett,

I came across your issue and Pull Request, as in the past you have contributed to the Upplication/Amazon-S3-FileSystem-NIO2 project. Would it be fine for you if we applied the changes of your Pull Request to our project?

Thanks in advance.
cc @carlspring

@thomascollett
Copy link

@ptirador @carlspring please go ahead

@carlspring
Copy link
Owner

Thanks for confirming, @thomascollett !

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 a pull request may close this issue.

3 participants