Move environment-variable checking into fs-mistrust
- Jul 19, 2022
-
-
Nick Mathewson authored -
These aren't user facing comments, but getting them right will help us write better changelogs.
-
Nick Mathewson authored
This logic can now be adjusted via the config object so that it does its own overriding by looking at the environment as appropriate. Removing these methods helps simplify the code a bit. Enabled by #483.
-
Nick Mathewson authored
Now that configuring the environment variables related to fs permissions works properly, we don't need to use the "override" feature any more: we can just add the option to the configuration when appropriate. With this design, `--disable-fs-permission-checks` is now mostly an alias for `--option storage.permissions.dangerously_trust_everyone=true` Enabled by #483.
-
Nick Mathewson authored
The variable is now handled when building the configuration, and no longer needs to be special-cased. Closes #483.
-
Nick Mathewson authored
By default we look at `$FS_MISTRUST_DISABLE_PERMISSIONS_CHECKS`. Optionally, the user can provide another variable as well, or disable looking at the environment entirely.
-
