framework icon indicating copy to clipboard operation
framework copied to clipboard

telemetry commands

Open mbostock opened this issue 2 years ago • 4 comments

Can we add observable telemetry enable and observable telemetry disable commands to enable and disable telemetry (in addition to the OBSERVABLE_TELEMETRY_DISABLE environment variable?

mbostock avatar Jan 11 '24 23:01 mbostock

observable telemetry remove-all-history?

Fil avatar Jan 12 '24 11:01 Fil

It’d be possible to do a remove if they sent their device id. Supposedly the uuidv4 Node crypto generates is “cryptographically pseudorandom” so it should be convincing if you can tell us one. I’d be kinda wary of someone abusing a remove api to spam us with lots of generated uuids though. I guess we could rate limit it or if you pass a device id that’s not found it exponentially increases your wait time to try again.

visnup avatar Jan 12 '24 15:01 visnup

@Fil that sounds like a separate feature request that should be tracked and discussed separately. This issue is about exposing functionality we already have through commands (rather than environment variables).

mbostock avatar Jan 16 '24 17:01 mbostock

probably a related command to tag the current user as a developer and send that extra flag along, to split ourselves out in the data. Similar to HOMEBREW_DEVELOPER.

visnup avatar Jan 26 '24 17:01 visnup