ahkamel commented on code in PR #1236:
URL: https://github.com/apache/struts/pull/1236#discussion_r2797712215


##########
core/src/main/java/org/apache/struts2/util/ProxyUtil.java:
##########
@@ -65,15 +66,18 @@ public class ProxyUtil {
      * object as fallback; never {@code null})
      */
     public static Class<?> ultimateTargetClass(Object candidate) {
-        Class<?> result = null;
-        if (isSpringAopProxy(candidate))
-            result = springUltimateTargetClass(candidate);
-
-        if (result == null) {
-            result = candidate.getClass();
-        }
-
-        return result;
+        return targetClassCache.computeIfAbsent(candidate, k -> {

Review Comment:
   @kusalk, Thanks for the update. I'm not sure when this release will be 
available, while upgrading to this Struts version is urgent for us because it 
includes an important security fix. Due to the current bug, we are experiencing 
OOM issues in our UAT environment on four nodes, approximately half an hour 
after deployment



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to