feat: add telemetry enrichment to StartRequest

Author: spikecurtis

Tests.Vpn.Service/TelemetryEnricherTest.cs

@@ -0,0 +1,34 @@
+using Coder.Desktop.Vpn.Proto;
+using Coder.Desktop.Vpn.Service;
+
+namespace Coder.Desktop.Tests.Vpn.Service;
+
+[TestFixture]
+public class TelemetryEnricherTest
+{
+    [Test]
+    public void EnrichStartRequest()
+    {
+        var req = new StartRequest
+        {
+            CoderUrl = "https://coder.example.com",
+        };
+        var enricher = new TelemetryEnricher();
+        req = enricher.EnrichStartRequest(req);
+
+        // quick sanity check that non-telemetry fields aren't lost or overwritten
+        Assert.That(req.CoderUrl, Is.EqualTo("https://coder.example.com"));
+
+        Assert.That(req.DeviceOs, Is.EqualTo("Windows"));
+        // seems that test assemblies always set 1.0.0.0
+        Assert.That(req.CoderDesktopVersion, Is.EqualTo("1.0.0.0"));
+        Assert.That(req.DeviceId, Is.Not.Empty);
+        var deviceId = req.DeviceId;
+
+        // deviceId is different on different machines, but we can test that
+        // each instance of the TelemetryEnricher produces the same value.
+        enricher = new TelemetryEnricher();
+        req = enricher.EnrichStartRequest(new StartRequest());
+        Assert.That(req.DeviceId, Is.EqualTo(deviceId));
+    }
+}

Vpn.Proto/RpcVersion.cs

@@ -5,7 +5,7 @@ namespace Coder.Desktop.Vpn.Proto;
 /// </summary>
 public class RpcVersion
 {
-    public static readonly RpcVersion Current = new(1, 0);
+    public static readonly RpcVersion Current = new(1, 1);
 
     public ulong Major { get; }
     public ulong Minor { get; }