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

UID and GID work correctly in latest; appear to be ignored in recent tags #24

Closed
akhayyat opened this issue Mar 15, 2022 · 3 comments
Closed
Assignees

Comments

@akhayyat
Copy link

Using the latest tag, the UID and GID environment variables behave correctly: setting them allows only the specified user to access the mounted directory.

However, using any of the other recent tags, e.g. 1.91, 1.90, 1.89, etc., the UID and GID environment variables do nothing at all: the user whose UID and GID are specified will be granted access to the mounted directory only if the allow_other option is used. Otherwise, it gets a "permission denied" error.

BTW, which s3fs version is used in the latest tag?

@efrecon efrecon self-assigned this Mar 16, 2022
@efrecon
Copy link
Owner

efrecon commented Mar 16, 2022

latest is hopelessly outdated (despite the name). But I think this is all because the version of latest was running s3fs as the user, which seems to have disappeared. I will have to dig in the commit history to understand why and when and return to you. The mount command should look like the following one:

su - $RUN_AS -c "s3fs $DEBUG_OPTS ${S3FS_ARGS} \
    -o passwd_file=${AWS_S3_AUTHFILE} \
    -o url=${AWS_S3_URL} \
    -o uid=$UID \
    -o gid=$GID \
    ${AWS_S3_BUCKET} ${DEST}"

I haven't got time right now, but you are welcome to give it a try in a local copy and tell me if that works better.

@efrecon
Copy link
Owner

efrecon commented Mar 16, 2022

I fixed it through 96052ad. Wait until the builds have finished and tell me if it works. I am closing this now, but can reopen on problems.

@efrecon efrecon closed this as completed Mar 16, 2022
@akhayyat
Copy link
Author

Seems to be working correctly now. Thanks!

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

No branches or pull requests

2 participants