This is an automated email from the ASF dual-hosted git repository. kusal pushed a commit to branch WW-5343-sec-extend in repository https://gitbox.apache.org/repos/asf/struts.git
commit b518635e2e7b1f56d1b837611f85938fff138dbb Author: Kusal Kithul-Godage <g...@kusal.io> AuthorDate: Wed Nov 15 00:24:39 2023 +1100 WW-5343 Update OgnlUtil#createDefaultContext to utilise SecurityMemberAccess bean --- core/src/main/java/com/opensymphony/xwork2/ognl/OgnlUtil.java | 11 +---------- 1 file changed, 1 insertion(+), 10 deletions(-) diff --git a/core/src/main/java/com/opensymphony/xwork2/ognl/OgnlUtil.java b/core/src/main/java/com/opensymphony/xwork2/ognl/OgnlUtil.java index bbcf3bdff..55b27b0e2 100644 --- a/core/src/main/java/com/opensymphony/xwork2/ognl/OgnlUtil.java +++ b/core/src/main/java/com/opensymphony/xwork2/ognl/OgnlUtil.java @@ -863,8 +863,7 @@ public class OgnlUtil { resolver = container.getInstance(CompoundRootAccessor.class); } - SecurityMemberAccess memberAccess = new SecurityMemberAccess(allowStaticFieldAccess); - memberAccess.disallowProxyMemberAccess(disallowProxyMemberAccess); + SecurityMemberAccess memberAccess = container.getInstance(SecurityMemberAccess.class); if (devMode) { if (!warnReported.get()) { @@ -875,14 +874,6 @@ public class OgnlUtil { memberAccess.useExcludedPackageNamePatterns(devModeExcludedPackageNamePatterns); memberAccess.useExcludedPackageNames(devModeExcludedPackageNames); memberAccess.useExcludedPackageExemptClasses(devModeExcludedPackageExemptClasses); - } else { - memberAccess.useExcludedClasses(getExcludedClasses()); - memberAccess.useExcludedPackageNamePatterns(getExcludedPackageNamePatterns()); - memberAccess.useExcludedPackageNames(getExcludedPackageNames()); - memberAccess.useExcludedPackageExemptClasses(getExcludedPackageExemptClasses()); - memberAccess.useEnforceAllowlistEnabled(isEnforceAllowlistEnabled()); - memberAccess.useAllowlistClasses(getAllowlistClasses()); - memberAccess.useAllowlistPackageNames(getAllowlistPackageNames()); } return Ognl.createDefaultContext(root, memberAccess, resolver, defaultConverter);