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
caddy: disable telemetry #63697
caddy: disable telemetry #63697
Conversation
What's the reason for disabling telemetry? |
@kalbasit Do we not disable telemetry as a matter of policy? It's disabled in firefox, and searching the repo I see that it's disabled in Powershell and TimescaleDB also. |
pkgs/servers/caddy/default.nix
Outdated
|
||
preBuild = '' | ||
substituteInPlace caddy/caddymain/run.go \ | ||
--replace "var EnableTelemetry = true" "var EnableTelemetry = false" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you make this a patch instead?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
also nitpick: var EnableTelemetry = false
is the same as var EnableTelemetry bool
.
Making a software not transmitting information to third-party services by default is a good practice, we follow in NixOS. |
c09d8a3
to
675a145
Compare
The recommended way by upstream to disable telemetry is to create an new main module: https://github.com/caddyserver/caddy#build. Since we also need this to package caddy modules in the future, I've rewritten this PR to overwrite the default main module with a custom one to disable the telemetry. Also rebased onto the current master. You can check if telemetry is disabled by checking if |
Co-authored-by: Franz Pletz <fpletz@fnordicwalking.de>
675a145
to
b53710e
Compare
@fpletz it works! thanks a lot for updating it |
Motivation for this change
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)