now it is connected top to bottom. tests only remain

tools/test-proxy/Azure.Sdk.Tools.TestProxy/Admin.cs

@@ -66,7 +66,7 @@ public async Task AddTransform()
         }
 
         [HttpPost]
-        public void RemoveSanitizers([FromBody]RemoveSanitizerList sanitizerList)
+        public async Task RemoveSanitizers([FromBody]RemoveSanitizerList sanitizerList)
         {
             DebugLogger.LogAdminRequestDetails(_logger, Request);
             var recordingId = RecordingHandler.GetHeader(Request, "x-recording-id", allowNulls: true);
@@ -90,6 +90,14 @@ public void RemoveSanitizers([FromBody]RemoveSanitizerList sanitizerList)
                     + string.Join("\n", exceptionsList);
                 throw new HttpException(HttpStatusCode.BadRequest, varExceptionMessage);
             }
+            else
+            {
+                var json = JsonSerializer.Serialize(new { RemovedSanitizers = removedSanitizers });
+                Response.ContentType = "application/json";
+                Response.ContentLength = json.Length;
+
+                await Response.WriteAsync(json);
+            }
         }
 
         [HttpPost]

tools/test-proxy/Azure.Sdk.Tools.TestProxy/Common/SanitizerDictionary.cs

@@ -58,6 +58,9 @@ public SanitizerDictionary() {
                 )
             };
 
+        /// <summary>
+        /// Used to update the session sanitizers to their default configuration.
+        /// </summary>
         public void ResetSessionSanitizers()
         {
             var expectedSanitizers = DefaultSanitizerList;
@@ -86,11 +89,20 @@ public List<RecordedTestSanitizer> GetSanitizers(ModifiableRecordSession session
             return GetRegisteredSanitizers(session).Select(x => x.Sanitizer).ToList();
         }
 
+        /// <summary>
+        /// Gets a list of sanitizers that should be applied for the session level.
+        /// </summary>
+        /// <returns></returns>
         public List<RecordedTestSanitizer> GetSanitizers()
         {
             return GetRegisteredSanitizers().Select(x => x.Sanitizer).ToList();
         }
 
+        /// <summary>
+        /// Get the set of registered sanitizers for a specific recording or playback session.
+        /// </summary>
+        /// <param name="session"></param>
+        /// <returns></returns>
         public List<RegisteredSanitizer> GetRegisteredSanitizers(ModifiableRecordSession session)
         {
             var sanitizers = new List<RegisteredSanitizer>();
@@ -105,6 +117,10 @@ public List<RegisteredSanitizer> GetRegisteredSanitizers(ModifiableRecordSession
             return sanitizers;
         }
 
+        /// <summary>
+        /// Gets the set of registered sanitizers for the session level.
+        /// </summary>
+        /// <returns></returns>
         public List<RegisteredSanitizer> GetRegisteredSanitizers()
         {
             var sanitizers = new List<RegisteredSanitizer>();
@@ -136,7 +152,7 @@ private bool _register(RecordedTestSanitizer sanitizer, string id)
         /// Ensuring that session level sanitizers can be identified internally
         /// </summary>
         /// <param name="sanitizer"></param>
-        /// <returns></returns>
+        /// <returns>The Id of the newly registered sanitizer.</returns>
         /// <exception cref="HttpException"></exception>
         public string Register(RecordedTestSanitizer sanitizer)
         {
@@ -155,7 +171,7 @@ public string Register(RecordedTestSanitizer sanitizer)
         /// </summary>
         /// <param name="session"></param>
         /// <param name="sanitizer"></param>
-        /// <returns></returns>
+        /// <returns>The Id of the newly registered sanitizer.</returns>
         /// <exception cref="HttpException"></exception>
         public string Register(ModifiableRecordSession session, RecordedTestSanitizer sanitizer)
         {
@@ -171,6 +187,12 @@ public string Register(ModifiableRecordSession session, RecordedTestSanitizer sa
             return string.Empty;
         }
 
+        /// <summary>
+        /// Removes a sanitizer from the global session set.
+        /// </summary>
+        /// <param name="sanitizerId"></param>
+        /// <returns></returns>
+        /// <exception cref="HttpException"></exception>
         public string Unregister(string sanitizerId)
         {
             if (SessionSanitizers.Contains(sanitizerId))
@@ -182,6 +204,13 @@ public string Unregister(string sanitizerId)
             throw new HttpException(System.Net.HttpStatusCode.BadRequest, $"The requested sanitizer for removal \"{sanitizerId}\" is not active at the session level.");
         }
 
+        /// <summary>
+        /// Removes  a sanitizer from a specific recording or playback session.
+        /// </summary>
+        /// <param name="sanitizerId"></param>
+        /// <param name="session"></param>
+        /// <returns></returns>
+        /// <exception cref="HttpException"></exception>
         public string Unregister(string sanitizerId, ModifiableRecordSession session)
         {
             if (session.AppliedSanitizers.Contains(sanitizerId))
@@ -204,13 +233,12 @@ public void Cleanup(ModifiableRecordSession session)
                 Sanitizers.TryRemove(sanitizerId, out var RemovedSanitizer);
             }
         }
+
+        /// <summary>
+        /// Not publically available via an API Route, but used to remove all of the active default session sanitizers.
+        /// </summary>
         public void Clear()
         {
-            foreach(var sanitizerId in SessionSanitizers)
-            {
-                Sanitizers.TryRemove(sanitizerId.ToString(), out var RemovedSanitizer);
-            }
-
             SessionSanitizers.Clear();
         }
     }