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

Set telemetry to disabled by default #4452

Closed

Conversation

jonvargason
Copy link

@jonvargason jonvargason commented Mar 3, 2025

Description

Disabling telemetry by default. Why: Some users and corporations may not want to have an opt-out model but would rather have an opt-in to avoid accidentally sending potentially company sensitive data by accident upon install. Full disclosure I don't believe anything this plugin sends is sensitive but companies are very concerned about data egress. I personally think we should enable it for academic and personal usage but enterprise adoption - this may be frowned on or likely would be blocked anyway by firewalls.

Checklist

  • The relevant docs, if any, have been updated or created
  • [] The relevant tests, if any, have been updated or created

Screenshots

image

image

After install:
image

Testing instructions

I'll be honest and say I could not get this to build on my work network but I suspect it was a proxy thing and VS issue as even though I made what I believe are the correct modifications and pointed to my instance of VS 2022 it still failed on sqlite and I am not sure why. I was able to pull the pipeline build at https://github.com/continuedev/continue/actions/runs/13640223539/job/38128264263?pr=4452.

I believe to test this you basically follow the instructions on contributing.

  1. Make sure you have the correct version of npm (v20.11.0)
  2. Pull down my PR
  3. Install all dependencies in VS Code
  4. Build and Run Extension locally
  5. Validate Telemetry is disabled by default (settings -> unchecked Telemetry Enabled)

Alternatively if you want to skip the build like I did locally:

  1. Grab that binary at https://github.com/continuedev/continue/actions/runs/13640223539/job/38128264263?pr=4452.
  2. Then Open VS Code and Install VSIX
  3. Validate Telemetry is disabled by default (settings -> unchecked Telemetry Enabled)

Copy link

netlify bot commented Mar 3, 2025

Deploy Preview for continuedev ready!

Name Link
🔨 Latest commit 9757143
🔍 Latest deploy log https://app.netlify.com/sites/continuedev/deploys/67c628643ebf190008571e13
😎 Deploy Preview https://deploy-preview-4452--continuedev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@sestinj
Copy link
Contributor

sestinj commented Mar 8, 2025

@jonvargason I appreciate you taking the time to make a quality PR here. At least as of now, opt-out telemetry is not something that we are looking to change, as it's a core part of our being able to understand and improve the product for all users. We aim to make sure that it's perfectly clear how to turn off telemetry for those who need to, and have even added a more easily accessible toggle in the sidebar UI recently. And I can confirm that, as you noted, none of the telemetry is sensitive: no code, no prompts, no personal information are ever sent.

However I can understand the tough situation where you might not reliably be able to have every new employee who downloads Continue remember immediately to turn it off. Depending on what sort of environment Continue is being used in we may be able to come to a creative solution for this. Some examples that I can think of immediately:

  • If hub.continue.dev is an option for you, you would be able to centrally turn off telemetry for all signed in users in your org
  • If you have the ability to manage files on computers, a configuration file that disables telemetry could be placed there
  • This traffic could be configured to be blocked at the network level, which provides the most possible security

Feel free to reach out if you want to discuss any of these possibilities

@sestinj sestinj closed this Mar 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants