[WW-5311] NamedVariablePatternMatcher throws an IllegalArgumentException when named variable is not the last part of the sequence

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 6.1.1, 6.1.2
• Fix Version/s: 6.2.0
• Component/s: Core
• Labels: regresion

Language

Java

• Assignee: Unassigned
• Reporter: David Vanacker
• Created: 15/May/23 23:47
• Updated: 16/Aug/23 20:07
• Resolved: 16/May/23 18:49

Description

This following snippet works as expected on Struts 6.0.3, but fails on 6.1.x.
 

import org.junit.Test;

import com.opensymphony.xwork2.util.NamedVariablePatternMatcher;

public class NamedVariablePatternMatcherTest {

    @Test
    public void test() {
        new NamedVariablePatternMatcher().compilePattern("/{first}/two/{third}");
        new NamedVariablePatternMatcher().compilePattern("/first/two/{third}");
        new NamedVariablePatternMatcher().compilePattern("/first/{two}/{third}");
        new NamedVariablePatternMatcher().compilePattern("/{first}/{two}/third"); // java.lang.IllegalArgumentException: Missing openning '{' in [/ {first}/{two}/third]!         
        new NamedVariablePatternMatcher().compilePattern("/{first}/two"); //java.lang.IllegalArgumentException: Missing openning '{' in [/{first}
/two]!     }
} 

 

 
 
 
 

Issue Links

• is caused by: WW-3529 NamedVariablePatternMatcher does not properly escape characters
• is fixed by: WW-5309 NamedVariablePatternMatcher throws when pattern begins with a variable
• relates to: WW-5309 NamedVariablePatternMatcher throws when pattern begins with a variable