Do not call End/Abort on disconnect #430 (#432)

aspnet · Oct 25, 2021 · cc2ae7e · cc2ae7e
1 parent 412e649
commit cc2ae7e
Showing 1 changed file with 21 additions and 17 deletions.
diff --git a/src/Microsoft.Owin.Host.HttpListener/RequestProcessing/OwinHttpListenerContext.cs b/src/Microsoft.Owin.Host.HttpListener/RequestProcessing/OwinHttpListenerContext.cs
@@ -62,22 +62,7 @@ internal void End(Exception ex)
         {
             if (ex != null)
             {
-                // TODO: LOG
-                // Lazy initialized
-                if (_cts != null)
-                {
-                    try
-                    {
-                        _cts.Cancel();
-                    }
-                    catch (ObjectDisposedException)
-                    {
-                    }
-                    catch (AggregateException)
-                    {
-                        // TODO: LOG
-                    }
-                }
+                CancelDisconnectToken();
             }
 
             End();
@@ -99,7 +84,26 @@ internal void End()
         private static void SetDisconnected(object state)
         {
             var context = (OwinHttpListenerContext)state;
-            context.End(new HttpListenerException(Constants.ErrorConnectionNoLongerValid));
+            context.CancelDisconnectToken();
+        }
+
+        private void CancelDisconnectToken()
+        {
+            // Lazy initialized
+            if (_cts != null)
+            {
+                try
+                {
+                    _cts.Cancel();
+                }
+                catch (ObjectDisposedException)
+                {
+                }
+                catch (AggregateException)
+                {
+                    // TODO: LOG
+                }
+            }
         }
 
         public void Dispose()