Skip to content

Use 0600 for ~/.config/safeeyes/safeeyes.json instead of 0666#817

Merged
archisman-panigrahi merged 1 commit intoslgobinath:masterfrom
hartwork:fix-resetconfig-permissions
Jan 17, 2026
Merged

Use 0600 for ~/.config/safeeyes/safeeyes.json instead of 0666#817
archisman-panigrahi merged 1 commit intoslgobinath:masterfrom
hartwork:fix-resetconfig-permissions

Conversation

@hartwork
Copy link
Copy Markdown
Contributor

Started out as 777 in 4939d1f, was reduced to 666 in 1d3f804 and is still more than I would expect. Am I missing something?

@archisman-panigrahi
Copy link
Copy Markdown
Collaborator

Can it cause any problems for users who will eventually upgrade?

@hartwork
Copy link
Copy Markdown
Contributor Author

One could argue that the home directory is not meant to be readable by anyone and that if that is the case, that's a bug on its own and then the issue here does not need fixing.
One could argue that no file in the home directory should ever be this widely accessible.
One could argue that I will not be the last one running into this wondering why the file has that loose permissions.
This pull request ignores argument (1) and improves on (2) and (3).
I don't expect any breakage and will be curious to learn about detail if I end up being wrong about it.
What do you think?

@archisman-panigrahi
Copy link
Copy Markdown
Collaborator

There should not be an issue for the native app. My only concern is whether the flatpak/nixpkg may stop working, but I will give this a try with flatpak.

@archisman-panigrahi
Copy link
Copy Markdown
Collaborator

Seems to work fine in flatpak

@archisman-panigrahi archisman-panigrahi merged commit f305fed into slgobinath:master Jan 17, 2026
6 checks passed
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 this pull request may close these issues.

2 participants