Feedback.

Author: ilonatommy

src/Components/Components/src/NavigationManager.cs

@@ -204,7 +204,10 @@ public virtual void Refresh(bool forceReload = false)
     /// <summary>
     /// Handles setting the NotFound state.
     /// </summary>
-    protected virtual void NotFoundCore() => throw new NotImplementedException();
+    protected virtual void NotFoundCore()
+    {
+        _notFound?.Invoke(this, new EventArgs());
+    }
 
     /// <summary>
     /// Called to initialize BaseURI and current URI before these values are used for the first time.
@@ -337,21 +340,6 @@ protected void NotifyLocationChanged(bool isInterceptedLink)
         }
     }
 
-    /// <summary>
-    /// Triggers the <see cref="NotFound"/> event with the current URI value.
-    /// </summary>
-    protected void NotifyNotFound()
-    {
-        try
-        {
-            _notFound?.Invoke(this, new EventArgs());
-        }
-        catch (Exception ex)
-        {
-            throw new NotFoundRenderingException("An exception occurred while dispatching a NotFound event.", ex);
-        }
-    }
-
     /// <summary>
     /// Notifies the registered handlers of the current location change.
     /// </summary>

src/Components/Components/src/NotFoundRenderingException.cs

@@ -2,9 +2,6 @@
 Microsoft.AspNetCore.Components.IEndpointHtmlRenderer
 Microsoft.AspNetCore.Components.IEndpointHtmlRenderer.SetNotFoundResponse() -> void
 Microsoft.AspNetCore.Components.NavigationManager.NotFoundEvent -> System.EventHandler<System.EventArgs!>!
-Microsoft.AspNetCore.Components.NavigationManager.NotifyNotFound() -> void
-Microsoft.AspNetCore.Components.NotFoundRenderingException
-Microsoft.AspNetCore.Components.NotFoundRenderingException.NotFoundRenderingException(string! message, System.Exception! innerException) -> void
 Microsoft.AspNetCore.Components.Routing.IHostEnvironmentNavigationManager.Initialize(string! baseUri, string! uri, Microsoft.AspNetCore.Components.IEndpointHtmlRenderer! renderer) -> void
 virtual Microsoft.AspNetCore.Components.NavigationManager.NotFound() -> void
 virtual Microsoft.AspNetCore.Components.NavigationManager.NotFoundCore() -> void

src/Components/Components/src/PublicAPI.Unshipped.txt

@@ -160,22 +160,6 @@ async Task RefreshAsync()
         }
     }
 
-    /// <inheritdoc />
-    protected override void NotFoundCore()
-    {
-        Log.RequestingNotFound(_logger);
-
-        try
-        {
-            NotifyNotFound();
-        }
-        catch (Exception ex)
-        {
-            Log.NotFoundRenderFailed(_logger, ex);
-            UnhandledException?.Invoke(this, ex);
-        }
-    }
-
     protected override void HandleLocationChangingHandlerException(Exception ex, LocationChangingContext context)
     {
         Log.NavigationFailed(_logger, context.TargetLocation, ex);
@@ -233,8 +217,5 @@ public static void RequestingNavigation(ILogger logger, string uri, NavigationOp
 
         [LoggerMessage(7, LogLevel.Debug, "Navigation stopped because the session ended when navigating to {Uri}", EventName = "NavigationStoppedSessionEnded")]
         public static partial void NavigationStoppedSessionEnded(ILogger logger, string uri);
-
-        [LoggerMessage(8, LogLevel.Error, "Failed to render NotFound", EventName = "NotFoundRenderFailed")]
-        public static partial void NotFoundRenderFailed(ILogger logger, Exception exception);
     }
 }

src/Components/Server/src/Circuits/RemoteNavigationManager.cs

@@ -85,19 +85,6 @@ public override void Refresh(bool forceReload = false)
         DefaultWebAssemblyJSRuntime.Instance.InvokeVoid(Interop.Refresh, forceReload);
     }
 
-    /// <inheritdoc />
-    protected override void NotFoundCore()
-    {
-        try
-        {
-            NotifyNotFound();
-        }
-        catch (Exception ex)
-        {
-            Log.NotFoundRenderFailed(_logger, ex);
-        }
-    }
-
     protected override void HandleLocationChangingHandlerException(Exception ex, LocationChangingContext context)
     {
         Log.NavigationFailed(_logger, context.TargetLocation, ex);
@@ -113,8 +100,5 @@ private static partial class Log
 
         [LoggerMessage(2, LogLevel.Error, "Navigation failed when changing the location to {Uri}", EventName = "NavigationFailed")]
         public static partial void NavigationFailed(ILogger logger, string uri, Exception exception);
-
-        [LoggerMessage(3, LogLevel.Error, "Failed to render NotFound", EventName = "NotFoundRenderFailed")]
-        public static partial void NotFoundRenderFailed(ILogger logger, Exception exception);
     }
 }