Merge pull request #322 from microsoft/main

Fork Sync: Update from parent repository

Author: AdamL-Microsoft

src/ApiService/ApiService/Functions/ReproVmss.cs

@@ -1,4 +1,6 @@
 using System.Net;
+using System.Security.Cryptography;
+using System.Text;
 using Microsoft.Azure.Functions.Worker;
 using Microsoft.Azure.Functions.Worker.Http;
 
@@ -81,6 +83,14 @@ private async Async.Task<HttpResponseData> Post(HttpRequestData req) {
                 "repro_vm create");
         }
 
+        // we’d like to track the usage of this feature; 
+        // anonymize the user ID so we can distinguish multiple requests
+        {
+            var data = userInfo.OkV.UserInfo.ToString(); // rely on record ToString
+            var hash = Convert.ToBase64String(SHA256.HashData(Encoding.UTF8.GetBytes(data)));
+            _log.Event($"created repro VM, user distinguisher: {hash:Tag:UserHash}");
+        }
+
         var response = req.CreateResponse(HttpStatusCode.OK);
         await response.WriteAsJsonAsync(vm.OkV);
         return response;

src/ApiService/ApiService/Log.cs

@@ -12,7 +12,7 @@ namespace Microsoft.OneFuzz.Service;
 [InterpolatedStringHandler]
 public struct LogStringHandler {
 
-    private StringBuilder _builder;
+    private readonly StringBuilder _builder;
     private Dictionary<string, string>? _tags;
 
     public LogStringHandler(int literalLength, int formattedCount) {