Add option to disable Passenger telemetry #4347
Comments
|
Yea it seems like it should be disabled by default. Happy to look at a PR for the same. That said - I think you can probably disable this through https://osc.github.io/ood-documentation/latest/reference/files/nginx-stage-yml.html |
|
Thanks. I would prefer to add a separate variable to make it clear this option exists (rather than requiring the user to know what option to specify). What is the correct release path here? I deployed the changes to the PUN template to my installation, but now nginx_stage is complaining about the unknown option ('Warning: invalid configuration option "passenger_disable_anonymous_telemetry"') and skips it when I want to generate a new PUN config. |
|
Yea it has to be supported here upstream before we can support it in the automation. |
This reverts commit d10d642 to return to a clean PR upstream.
Hey
While debugging an issue during the Passenger startup I noticed that the Passenger telemetry is enabled by default and haven't found any option in OnDemand to disable it. I believe the user should have an option for that.
Probably the best way would be to add the "passenger_disable_anonymous_telemetry on;" variable to the nginx pun config of the user at /var/lib/ondemand-nginx/config/.conf.
This would probably mean adding a variable to the defaults which is then parsed inside the template for the user PUNs. The variable could also be used by the Ansible role.
I'm happy to try to provide the required changes as pull request if you are interested in this improvement.
The text was updated successfully, but these errors were encountered: