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

[FEATURE] Add ability to skip permission checks #25

Closed
edyounis opened this issue Apr 1, 2021 · 1 comment · Fixed by #30
Closed

[FEATURE] Add ability to skip permission checks #25

edyounis opened this issue Apr 1, 2021 · 1 comment · Fixed by #30
Assignees
Labels
enhancement New feature or request

Comments

@edyounis
Copy link

edyounis commented Apr 1, 2021

Is your feature request related to a problem? Please describe.
chown -R command can create long boot times (>1 day) with flattened encrypted directories, like cryfs

Describe the solution you'd like

  1. Add environment variable DROPBOX_SKIP_PERMISSIONS
  2. In the entrypoint script change:
# Change ownership to dropbox account on all working folders.
chown -R ${DROPBOX_UID}:${DROPBOX_GID} /opt/dropbox

to

# Change ownership to dropbox account on all working folders.
if [[ -z "$DROPBOX_SKIP_PERMISSIONS" ]]; then
    chown -R ${DROPBOX_UID}:${DROPBOX_GID} /opt/dropbox
else
    echo "Skipping permissions check, ensure the dropbox user owns all files."
fi

Additional context
N/A

@otherguy
Copy link
Owner

otherguy commented Jun 25, 2021

Hey @edyounis and thanks for your feature request! Sorry I'm only getting around to this now.

Version 1.9.0 now has a flag to do just that!

docker run --rm -it -e SKIP_SET_PERMISSIONS=true otherguy/dropbox:1.9.0

I would be grateful if you could try it out and let me know if this works for you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants