From af7c2b8b4744738ffd70c427406b6657638a700b Mon Sep 17 00:00:00 2001 From: Max Brunsfeld Date: Wed, 28 Sep 2022 15:21:49 -0700 Subject: [PATCH 1/2] Set minimum user id length in amplitude calls --- crates/client/src/telemetry.rs | 12 +++++++++++- 1 file changed, 11 insertions(+), 1 deletion(-) diff --git a/crates/client/src/telemetry.rs b/crates/client/src/telemetry.rs index 77aa308f30fadf586daff9ce5070c776f94f2a3d..8b7be5ba8089503be1763313695e4f88baee5504 100644 --- a/crates/client/src/telemetry.rs +++ b/crates/client/src/telemetry.rs @@ -52,6 +52,12 @@ lazy_static! { struct AmplitudeEventBatch { api_key: &'static str, events: Vec, + options: AmplitudeEventBatchOptions, +} + +#[derive(Serialize)] +struct AmplitudeEventBatchOptions { + min_id_length: usize, } #[derive(Serialize)] @@ -239,7 +245,11 @@ impl Telemetry { } } - let batch = AmplitudeEventBatch { api_key, events }; + let batch = AmplitudeEventBatch { + api_key, + events, + options: AmplitudeEventBatchOptions { min_id_length: 1 }, + }; json_bytes.clear(); serde_json::to_writer(&mut json_bytes, &batch)?; let request = From 0d3486ca82dc6812d05b3c6876240ee62cd1ab9d Mon Sep 17 00:00:00 2001 From: Max Brunsfeld Date: Wed, 28 Sep 2022 15:30:55 -0700 Subject: [PATCH 2/2] Remove TestTelemetry command --- crates/client/src/client.rs | 21 ++++----------------- 1 file changed, 4 insertions(+), 17 deletions(-) diff --git a/crates/client/src/client.rs b/crates/client/src/client.rs index 0670add1afd68f4963c57fc0f9c725b7c946bb30..b75be6230875b42608cdfa3a4d52028674d44ed9 100644 --- a/crates/client/src/client.rs +++ b/crates/client/src/client.rs @@ -15,11 +15,9 @@ use async_tungstenite::tungstenite::{ use db::Db; use futures::{future::LocalBoxFuture, FutureExt, SinkExt, StreamExt, TryStreamExt}; use gpui::{ - actions, - serde_json::{json, Value}, - AnyModelHandle, AnyViewHandle, AnyWeakModelHandle, AnyWeakViewHandle, AppContext, - AsyncAppContext, Entity, ModelContext, ModelHandle, MutableAppContext, Task, View, ViewContext, - ViewHandle, + actions, serde_json::Value, AnyModelHandle, AnyViewHandle, AnyWeakModelHandle, + AnyWeakViewHandle, AppContext, AsyncAppContext, Entity, ModelContext, ModelHandle, + MutableAppContext, Task, View, ViewContext, ViewHandle, }; use http::HttpClient; use lazy_static::lazy_static; @@ -56,7 +54,7 @@ lazy_static! { pub const ZED_SECRET_CLIENT_TOKEN: &str = "618033988749894"; -actions!(client, [Authenticate, TestTelemetry]); +actions!(client, [Authenticate]); pub fn init(client: Arc, cx: &mut MutableAppContext) { cx.add_global_action({ @@ -69,17 +67,6 @@ pub fn init(client: Arc, cx: &mut MutableAppContext) { .detach(); } }); - cx.add_global_action({ - let client = client.clone(); - move |_: &TestTelemetry, _| { - client.report_event( - "test_telemetry", - json!({ - "test_property": "test_value" - }), - ) - } - }); } pub struct Client {