Audit log exception check added

2 years ago · ea5c407b2a
2 changed files with 10 additions and 4 deletions
--- a/framework/src/Volo.Abp.AspNetCore.Mvc/Volo/Abp/AspNetCore/Mvc/Auditing/AbpAuditActionFilter.cs
+++ b/framework/src/Volo.Abp.AspNetCore.Mvc/Volo/Abp/AspNetCore/Mvc/Auditing/AbpAuditActionFilter.cs
@ -29,14 +29,17 @@ public class AbpAuditActionFilter : IAsyncActionFilter, IAbpFilter, ITransientDe
            {
                var result = await next();

-                if (result.Exception != null && !result.ExceptionHandled)
+                if (result.Exception != null && !auditLog!.Exceptions.Contains(result.Exception))
                {
                    auditLog!.Exceptions.Add(result.Exception);
                }
            }
            catch (Exception ex)
            {
-                auditLog!.Exceptions.Add(ex);
+                if (!auditLog!.Exceptions.Contains(ex))
+                {
+                    auditLog!.Exceptions.Add(ex);
+                }
                throw;
            }
            finally
--- a/framework/src/Volo.Abp.AspNetCore.Mvc/Volo/Abp/AspNetCore/Mvc/Auditing/AbpAuditPageFilter.cs
+++ b/framework/src/Volo.Abp.AspNetCore.Mvc/Volo/Abp/AspNetCore/Mvc/Auditing/AbpAuditPageFilter.cs
@ -34,14 +34,17 @@ public class AbpAuditPageFilter : IAsyncPageFilter, IAbpFilter, ITransientDepend
            {
                var result = await next();

-                if (result.Exception != null && !result.ExceptionHandled)
+                if (result.Exception != null && !auditLog!.Exceptions.Contains(result.Exception))
                {
                    auditLog!.Exceptions.Add(result.Exception);
                }
            }
            catch (Exception ex)
            {
-                auditLog!.Exceptions.Add(ex);
+                if (!auditLog!.Exceptions.Contains(ex))
+                {
+                    auditLog!.Exceptions.Add(ex);
+                }
                throw;
            }
            finally