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

Update Notifier feature can result in ' Error: EACCES: permission denied, mkdir '/.config' errors #693

Closed
lekkas opened this issue Oct 20, 2017 · 3 comments

Comments

@lekkas
Copy link
Contributor

lekkas commented Oct 20, 2017

  • resin-cli version: 6.7.0
  • Operating system and architecture: Ubuntu 16.04 base image

Possibly related snippets:

https://github.com/resin-io/resin-cli/blob/0b26fda89cdbf262c841b0753d3b8064b8c5514c/lib/utils/update.coffee#L25-L31

Also see https://www.npmjs.com/package/update-notifier#user-settings

@pimterry
Copy link
Contributor

We should make update notifier use RESINRC_DATA_DIRECTORY correctly, and make the CLI resilient to any update notifier errors too, so we ignore any kind of failures here (while still reporting them to sentry).

@pimterry
Copy link
Contributor

With #698 solved this is now much lower priority: these failures no longer break the CLI, and the update notifier can be disabled entirely with NO_UPDATE_NOTIFIER.

We should still fix this to obey RESINRC_DATA_DIRECTORY, so that users with non-writable ~/.config still get update notifications, but I'll drop the priority.

@pdcastro
Copy link
Contributor

pdcastro commented May 5, 2020

Closing this issue in favour of #1667 and #1669, and also because update-notifier has had a couple of major-version updates (now at 4.1.0) since this issue was created.

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

No branches or pull requests

3 participants