-
Notifications
You must be signed in to change notification settings - Fork 9
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
permissions are too open #103
Comments
Thanks for opening this issue! |
This is an anti-pattern since it stores your dae config file into nix store with global-readable permission. Please use the absolute path string as example shows. |
why it happen again when i use str ab path |
How do you configure it? |
If so it might be a literally |
{
config,
inputs,
lib,
pkgs,
...
}: {
environment.systemPackages = with inputs.daeuniverse.packages.x86_64-linux;
lib.optionals config.services.daed.enable [
dae
daed
];
services.dae = {
disableTxChecksumIpGeneric = false;
enable = true;
openFirewall = {
enable = true;
port = 12345;
};
configFile = "/home/solitudealma/nixos-config/nixos/_mixins/configs/dae/config.dae";
assets = with pkgs; [v2ray-geoip v2ray-domain-list-community];
};
services.daed = {
assetsPaths = with pkgs; [
"${v2ray-geoip}/share/v2ray/geoip.dat"
"${v2ray-domain-list-community}/share/v2ray/geosite.dat"
];
configDir = "/home/solitudealma/nixos-config/nixos/_mixins/configs/dae";
enable = true;
openFirewall = {
enable = true;
port = 12345;
};
};
# manual launch(geoip.dat,geosite.dat,config.dae needs to be in the same directory):
# `wget https://cdn.jsdelivr.net/gh/Loyalsoldier/geoip@release/geoip.dat`
# `wget wget https://cdn.jsdelivr.net/gh/Loyalsoldier/v2ray-rules-dat@release/geosite.dat`
# `dae run --disable-timestamp -c /path/to/config.dae`
} err:
|
Set permission just solve this.
Since we have no way to check the file permission at eval time. The file permission too open, as it reported at runtime. |
thx |
my config:
permission of configFile:
(change this in
/etc/nixos
will not affect that in/nix/store
)The text was updated successfully, but these errors were encountered: