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

Directories permissions on Windows #28

Open
darosior opened this issue Nov 19, 2020 · 2 comments
Open

Directories permissions on Windows #28

darosior opened this issue Nov 19, 2020 · 2 comments
Labels
enhancement New feature or request good first issue Good for newcomers High prio

Comments

@darosior
Copy link
Member

darosior commented Nov 19, 2020

revaultd is creating a directory (the "datadir") and some files (the database, the UNIX domain socket for RPC) on the host OS. We want to setup some basic OS permissions on these files (why would another user get allowed to modify our database, or use our RPC socket?). These permissions are set up at creation time in different places:

However, they are only currently setup for UNIX systems, and Windows is special-cased to setup no particular restrictions. We need to setup some permissions also on Windows, but first we need to figure out how :)

Some guidance on this issue:

  • It seems that the equivalent of permissions under Windows is to use ACLs. Maybe the stdlib has some way of abstracting it. Please don't introduce new dependencies just for this :).
  • To decode the UNIX permissions set in the links above, we want rw only for the owner of the file for the database and the RPC and rwx (still only for the owner) for the data directory.
@darosior darosior added the good first issue Good for newcomers label Dec 17, 2020
@darosior darosior added enhancement New feature or request High prio labels Feb 21, 2021
@darosior
Copy link
Member Author

@vaibhavgarg237
Copy link
Contributor

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers High prio
Projects
Development

No branches or pull requests

2 participants