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

Check directory permissions and throw an informative error when they are wrong #46

Open
scottillogical opened this issue Oct 28, 2019 · 0 comments

Comments

@scottillogical
Copy link

Thank you for this library! It has been working well for me/us for 3+ years

If you have a file you are going to go-update, and you run
sudo chown $DIR
Then run the code that runs update.Apply
You get failed to apply update.

If you then chown the directory, the update application will work.

To fix this, I think we need to add a step to verify the permissions of the parent directory for the target file before applying the update and return an informative error.

@scottillogical scottillogical changed the title returns only "failed to apply update" when directory perms are wrong Check directory permissions and throw an informative error when they are wrong Oct 28, 2019
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

1 participant