[MNG-8388] Fix escape characters being replaced to change the origina…

…l configuration (#1946) * [MNG-8388] Fix escape characters being replaced to change the original configuration Signed-off-by: crazyhzm <crazyhzm@apache.org> * Add unit test --------- Signed-off-by: crazyhzm <crazyhzm@apache.org> Co-authored-by: Guillaume Nodet <gnodet@gmail.com>
apache · Dec 5, 2024 · d6edb02 · d6edb02
1 parent 01e4725
commit d6edb02
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/impl/maven-impl/src/main/java/org/apache/maven/internal/impl/model/DefaultInterpolator.java b/impl/maven-impl/src/main/java/org/apache/maven/internal/impl/model/DefaultInterpolator.java
@@ -420,7 +420,7 @@ public static String unescape(@Nullable String val) {
         int escape = val.indexOf(ESCAPE_CHAR);
         while (escape >= 0 && escape < val.length() - 1) {
             char c = val.charAt(escape + 1);
-            if (c == '{' || c == '}' || c == ESCAPE_CHAR) {
+            if (c == '{' || c == '}') {
                 val = val.substring(0, escape) + val.substring(escape + 1);
             }
             escape = val.indexOf(ESCAPE_CHAR, escape + 1);

diff --git a/...aven-impl/src/test/java/org/apache/maven/internal/impl/model/DefaultInterpolatorTest.java b/...aven-impl/src/test/java/org/apache/maven/internal/impl/model/DefaultInterpolatorTest.java
@@ -98,6 +98,7 @@ void testSubstitutionEscape() {
         assertEquals("${a}", substVars("$\\{a${#}\\}", "b"));
         assertEquals("${a}", substVars("$\\{a\\}${#}", "b"));
         assertEquals("${a}", substVars("$\\{a\\}", "b"));
+        assertEquals("\\\\", substVars("\\\\", "b"));
     }
 
     @Test