Move `AlwaysLogSelectors` to the top.

framework/src/Volo.Abp.AspNetCore.SignalR/Volo/Abp/AspNetCore/SignalR/Auditing/AbpAuditHubFilter.cs

@@ -79,10 +79,6 @@ public class AbpAuditHubFilter : IHubFilter
     private async Task<bool> ShouldWriteAuditLogAsync(AuditLogInfo auditLogInfo, IServiceProvider serviceProvider, bool hasError)
     {
         var options = serviceProvider.GetRequiredService<IOptions<AbpAuditingOptions>>().Value;
-        if (options.AlwaysLogOnException && hasError)
-        {
-            return true;
-        }
 
         foreach (var selector in options.AlwaysLogSelectors)
         {
@@ -92,6 +88,11 @@ public class AbpAuditHubFilter : IHubFilter
             }
         }
 
+        if (options.AlwaysLogOnException && hasError)
+        {
+            return true;
+        }
+
         if (!options.IsEnabledForAnonymousUsers && !serviceProvider.GetRequiredService<ICurrentUser>().IsAuthenticated)
         {
             return false;

framework/src/Volo.Abp.AspNetCore/Volo/Abp/AspNetCore/Auditing/AbpAuditingMiddleware.cs

@@ -100,11 +100,6 @@ public class AbpAuditingMiddleware : IMiddleware, ITransientDependency
 
     private async Task<bool> ShouldWriteAuditLogAsync(AuditLogInfo auditLogInfo, HttpContext httpContext, bool hasError)
     {
-        if (AuditingOptions.AlwaysLogOnException && hasError)
-        {
-            return true;
-        }
-
         foreach (var selector in AuditingOptions.AlwaysLogSelectors)
         {
             if (await selector(auditLogInfo))
@@ -113,6 +108,11 @@ public class AbpAuditingMiddleware : IMiddleware, ITransientDependency
             }
         }
 
+        if (AuditingOptions.AlwaysLogOnException && hasError)
+        {
+            return true;
+        }
+
         if (!AuditingOptions.IsEnabledForAnonymousUsers && !CurrentUser.IsAuthenticated)
         {
             return false;

framework/src/Volo.Abp.Auditing/Volo/Abp/Auditing/AuditingInterceptor.cs

@@ -160,11 +160,6 @@ public class AuditingInterceptor : AbpInterceptor, ITransientDependency
         ICurrentUser currentUser,
         bool hasError)
     {
-        if (options.AlwaysLogOnException && hasError)
-        {
-            return true;
-        }
-
         foreach (var selector in options.AlwaysLogSelectors)
         {
             if (await selector(auditLogInfo))
@@ -173,6 +168,11 @@ public class AuditingInterceptor : AbpInterceptor, ITransientDependency
             }
         }
 
+        if (options.AlwaysLogOnException && hasError)
+        {
+            return true;
+        }
+
         if (!options.IsEnabledForAnonymousUsers && !currentUser.IsAuthenticated)
         {
             return false;