This is an automated email from the ASF dual-hosted git repository. lukaszlenart pushed a commit to branch WW-5837-ognl-3-4-17 in repository https://gitbox.apache.org/repos/asf/struts.git
commit 274dbf964e5479c5873c885dcda4280f14786f77 Author: Lukasz Lenart <[email protected]> AuthorDate: Wed Nov 4 07:38:30 2026 +0100 WW-5837 build(deps): upgrade OGNL to version 3.4.17 OGNL 3.4.17 matches property names exactly in OgnlRuntime.getWriteMethod, so the mis-cased setexcludedfieldstring no longer resolves for excludedFieldString. Update the fixture to expect no write method. Co-Authored-By: Claude Opus 5.5 <[email protected]> --- .../test/java/org/apache/struts2/ognl/OgnlSetPossiblePropertyTest.java | 2 +- pom.xml | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/core/src/test/java/org/apache/struts2/ognl/OgnlSetPossiblePropertyTest.java b/core/src/test/java/org/apache/struts2/ognl/OgnlSetPossiblePropertyTest.java index d9ae784fd..9e95cafbc 100644 --- a/core/src/test/java/org/apache/struts2/ognl/OgnlSetPossiblePropertyTest.java +++ b/core/src/test/java/org/apache/struts2/ognl/OgnlSetPossiblePropertyTest.java @@ -122,7 +122,7 @@ public class OgnlSetPossiblePropertyTest extends XWorkTestCase { Method writeMethod = OgnlRuntime.getWriteMethod(HolderWhoseFieldWithPublicWriteMethod.ExcludedField.class, "excludedFieldString"); vs.setValue("excludedField.excludedFieldString", "EXPLOITED"); - assertEquals("setexcludedfieldstring", writeMethod.getName()); + assertNull(writeMethod); assertNotEquals("EXPLOITED", holder.excludedField.excludedFieldString); } diff --git a/pom.xml b/pom.xml index 5caae7a5d..a66a1ccfd 100644 --- a/pom.xml +++ b/pom.xml @@ -127,7 +127,7 @@ <jaxb-impl.version>4.0.9</jaxb-impl.version> <log4j2.version>2.26.1</log4j2.version> <mockito.version>5.24.0</mockito.version> - <ognl.version>3.4.16</ognl.version> + <ognl.version>3.4.17</ognl.version> <spring.version>6.2.19</spring.version> <struts-annotations.version>2.0</struts-annotations.version> <tomcat-embed.version>10.1.59</tomcat-embed.version>
