-
Notifications
You must be signed in to change notification settings - Fork 5.9k
Telemetry data #1829
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
Comments
Yup it's the exact same data, we just fill in the telemetry endpoint with our own.
It looks like for some reason the configuration isn't being read or isn't being read properly. Even if the setting is false, it still reports it as being true. I don't see anything obviously wrong so we'll have to dig a bit. |
Thanks for reporting this! Broken configuration is a pretty big deal. I'll need to pay attention to how VS Code registers services when updating in the future so I don't miss changes in how things are initialized like this again. |
I've noticed that the server makes a lot of requests to
v1.telemetry.coder.com
. I'm curious what sort of data is being collected. It appears to be added here: https://github.com/cdr/code-server/blob/06b387fe98f93dceab989b4bdf1396c41035cb2a/ci/dev/vscode.patch#L2086 Is this simply redirecting data that Microsoft already collects in VSCode? I have Telemetry turned off in the settings, although I recognize that doesn't fully turn it off for VSCode either (at least, last I checked).I'm flagging this as a bug, because I have telemetry turned off and the application should respect this setting. In the last 7 days my DNS server has blocked
40,871
requests tov1.telemetry.coder.com
.(edit) It occours to me that I don't have
--disable-telemetry
passed, but have disbled it in the settings instead. Curious if these are expected to behave the same.code-server --version
:The text was updated successfully, but these errors were encountered: