-
Notifications
You must be signed in to change notification settings - Fork 79
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
Disable Microsoft telemetry #2882
Conversation
// --- Start Positron --- | ||
// 'default': true, | ||
'default': false, | ||
// --- End Positron --- |
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.
I feel like this one could be argued either way (keep or remove) but I would vote for removing.
// --- Start Positron --- | ||
// const endpointUrl = 'https://mobile.events.data.microsoft.com/OneCollector/1.0'; | ||
// const endpointHealthUrl = 'https://mobile.events.data.microsoft.com/ping'; | ||
const endpointUrl = 'https://mobile.events.0.0.0.0/OneCollector/1.0'; |
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.
While that is an invalid host, were we literally trying to use 0.0.0.0
to be IP of "this machine" or even an invalid IP, and if so, should we drop mobile.events. prefix? With telemetry disabled, I trust these URLs aren't actually used, though.
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.
Ah, that's probably better, yes
Intent
Addresses #2522
Approach
This PR takes a similar approach as VSCodium. I took a look at this info when deciding what to do/remove:
QA Notes
To see if there are other references to these servers, you can search for
data.microsoft.com
in the codebase.Also, you can try rebuilding Positron from scratch and then doing
rg --no-ignore -l "data.microsoft.com" .
from Positron's directory. You'll see a lot of stuff, but it is onlynode_modules
. If you dorg -l "data.microsoft.com" .
you'll only see the1dsAppender.ts
we have edited here in this PR.