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

feat/telemetry-off-default #3940

Open
InAnYan opened this issue Mar 5, 2025 · 0 comments
Open

feat/telemetry-off-default #3940

InAnYan opened this issue Mar 5, 2025 · 0 comments
Labels
enhancement New feature or request

Comments

@InAnYan
Copy link

InAnYan commented Mar 5, 2025

Is your feature request related to a problem? Please describe.

Telemetry is turned on by default. Questions and opinions on this topic could start a holy war topic, so I tried to separate subjective and objective problems of this decision:

  1. Objective:
  • What if user accidentally forgets to switch off telemetry?
  • What if user didn't even know that telemetry exists in the library?
  • What if you, developers, forget about writing some if statement for the telemetry? The very idea that you allow the existence of telemetry brings problems. Humans are not 100% ideal.
  1. Subjective
  • This is the first time I see something like that in libraries. And I personally don't like it. Why would I need to make extra steps to just use your library without any caveats?
  • There are many people for who privacy concerns are very important, who just dislike the idea of telemetry. They might find this shady.

Describe the solution you'd like
What about making telemetry turned off by default?

Describe alternatives you've considered

Of course, I understand!

  • You made this because you need telemetry. If for enabling telemetry, user had to set an environment variable, no one would spend time on doing this.
  • And, of course, I need to thank you that you do not force people to use telemetry, and that there is a way to turn it off. This is very good.

Additional context

Do you really need to include telemetry in a library?

I couldn't find a similar issue on this repo. Sorry, if I duplicated

@InAnYan InAnYan added the enhancement New feature or request label Mar 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant