Don't call HttpCloseUrlGroup if you didn't create it (#28336)

* Add created bool

* Be explicit in setting the flag

Author: shirhatti

src/Servers/HttpSys/src/NativeInterop/UrlGroup.cs

@@ -20,13 +20,15 @@ internal class UrlGroup : IDisposable
 
         private ServerSession _serverSession;
         private bool _disposed;
+        private bool _created;
 
         internal unsafe UrlGroup(ServerSession serverSession, ILogger logger)
         {
             _serverSession = serverSession;
             _logger = logger;
 
             ulong urlGroupId = 0;
+            _created = true;
             var statusCode = HttpApi.HttpCreateUrlGroup(
                 _serverSession.Id.DangerousGetServerSessionId(), &urlGroupId, 0);
 
@@ -44,6 +46,7 @@ internal unsafe UrlGroup(RequestQueue requestQueue, UrlPrefix url, ILogger logge
             _logger = logger;
 
             ulong urlGroupId = 0;
+            _created = false;
             var statusCode = HttpApi.HttpFindUrlGroupId(
                 url.FullPrefix, requestQueue.Handle, &urlGroupId);
 
@@ -141,14 +144,20 @@ public void Dispose()
 
             _disposed = true;
 
-            Debug.Assert(Id != 0, "HttpCloseUrlGroup called with invalid url group id");
+            if (_created)
+            {
 
-            uint statusCode = HttpApi.HttpCloseUrlGroup(Id);
+                Debug.Assert(Id != 0, "HttpCloseUrlGroup called with invalid url group id");
+
+                uint statusCode = HttpApi.HttpCloseUrlGroup(Id);
+
+                if (statusCode != UnsafeNclNativeMethods.ErrorCodes.ERROR_SUCCESS)
+                {
+                    _logger.LogError(LoggerEventIds.CloseUrlGroupError, "HttpCloseUrlGroup; Result: {0}", statusCode);
+                }
 
-            if (statusCode != UnsafeNclNativeMethods.ErrorCodes.ERROR_SUCCESS)
-            {
-                _logger.LogError(LoggerEventIds.CloseUrlGroupError, "HttpCloseUrlGroup; Result: {0}" , statusCode);
             }
+
             Id = 0;
         }