From aa9bf83c6d0be0c40e868e085dfd8e809afa4e91 Mon Sep 17 00:00:00 2001 From: Josh Cummings Date: Fri, 31 May 2024 12:34:33 -0600 Subject: [PATCH] Polish Exception Handling Issue gh-15093 --- .../DefaultMethodSecurityExpressionHandler.java | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/core/src/main/java/org/springframework/security/access/expression/method/DefaultMethodSecurityExpressionHandler.java b/core/src/main/java/org/springframework/security/access/expression/method/DefaultMethodSecurityExpressionHandler.java index 8d9175bf5b..2460bb6056 100644 --- a/core/src/main/java/org/springframework/security/access/expression/method/DefaultMethodSecurityExpressionHandler.java +++ b/core/src/main/java/org/springframework/security/access/expression/method/DefaultMethodSecurityExpressionHandler.java @@ -158,10 +158,10 @@ public class DefaultMethodSecurityExpressionHandler extends AbstractSecurityExpr filterTarget.addAll(retain); return filterTarget; } - catch (UnsupportedOperationException unsupportedOperationException) { - this.logger.debug(LogMessage.format( + catch (UnsupportedOperationException readonly) { + this.logger.trace(LogMessage.format( "Collection threw exception: %s. Will return a new instance instead of mutating its state.", - unsupportedOperationException.getMessage())); + readonly.getMessage())); return retain; } } @@ -204,10 +204,10 @@ public class DefaultMethodSecurityExpressionHandler extends AbstractSecurityExpr filterTarget.putAll(retain); return filterTarget; } - catch (UnsupportedOperationException unsupportedOperationException) { - this.logger.debug(LogMessage.format( + catch (UnsupportedOperationException readonly) { + this.logger.trace(LogMessage.format( "Map threw exception: %s. Will return a new instance instead of mutating its state.", - unsupportedOperationException.getMessage())); + readonly.getMessage())); return retain; } }