You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@groovy.apache.org by pa...@apache.org on 2026/06/01 03:39:20 UTC
(groovy) branch GROOVY_5_0_X updated (801a5c8780 -> ac040ce979)
This is an automated email from the ASF dual-hosted git repository.
paulk-asert pushed a change to branch GROOVY_5_0_X
in repository https://gitbox.apache.org/repos/asf/groovy.git
from 801a5c8780 GROOVY-12056: Bump javaparser to 3.28.2
new 81575187cb reject backslash in grape ivy coordinate validation
new 572ddf5f1c use Locale.ROOT in CharSequenceValue.toEnum case fallback
new ac040ce979 use Locale.ROOT in StringGroovyMethods.containsIgnoreCase
The 3 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails. The revisions
listed as "add" were already present in the repository and have only
been added to this reference.
Summary of changes:
src/main/groovy/groovy/grape/GrapeIvy.groovy | 2 +-
src/main/java/org/codehaus/groovy/runtime/StringGroovyMethods.java | 3 ++-
.../main/java/org/apache/groovy/json/internal/CharSequenceValue.java | 3 ++-
3 files changed, 5 insertions(+), 3 deletions(-)
(groovy) 03/03: use Locale.ROOT in StringGroovyMethods.containsIgnoreCase
Posted by pa...@apache.org.
This is an automated email from the ASF dual-hosted git repository.
paulk-asert pushed a commit to branch GROOVY_5_0_X
in repository https://gitbox.apache.org/repos/asf/groovy.git
commit ac040ce9794245180dc3f9005fe6a6385a84339b
Author: netliomax25-code <ne...@gmail.com>
AuthorDate: Thu May 28 12:38:56 2026 +0530
use Locale.ROOT in StringGroovyMethods.containsIgnoreCase
---
src/main/java/org/codehaus/groovy/runtime/StringGroovyMethods.java | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/src/main/java/org/codehaus/groovy/runtime/StringGroovyMethods.java b/src/main/java/org/codehaus/groovy/runtime/StringGroovyMethods.java
index 7cdadcdf08..5b006db751 100644
--- a/src/main/java/org/codehaus/groovy/runtime/StringGroovyMethods.java
+++ b/src/main/java/org/codehaus/groovy/runtime/StringGroovyMethods.java
@@ -48,6 +48,7 @@ import java.util.HashSet;
import java.util.Iterator;
import java.util.List;
import java.util.ListIterator;
+import java.util.Locale;
import java.util.Map;
import java.util.NoSuchElementException;
import java.util.Optional;
@@ -4242,6 +4243,6 @@ public class StringGroovyMethods extends DefaultGroovyMethodsSupport {
if (searchString == null || searchString.length() == 0 || self.length() < searchString.length())
return false;
- return self.toString().toLowerCase().contains(searchString.toString().toLowerCase());
+ return self.toString().toLowerCase(Locale.ROOT).contains(searchString.toString().toLowerCase(Locale.ROOT));
}
}
(groovy) 01/03: reject backslash in grape ivy coordinate validation
Posted by pa...@apache.org.
This is an automated email from the ASF dual-hosted git repository.
paulk-asert pushed a commit to branch GROOVY_5_0_X
in repository https://gitbox.apache.org/repos/asf/groovy.git
commit 81575187cbc194a5d6fc0e52d0673d75b5aa8279
Author: netliomax25-code <ne...@gmail.com>
AuthorDate: Sun May 31 22:29:42 2026 +0530
reject backslash in grape ivy coordinate validation
---
src/main/groovy/groovy/grape/GrapeIvy.groovy | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/main/groovy/groovy/grape/GrapeIvy.groovy b/src/main/groovy/groovy/grape/GrapeIvy.groovy
index b2d6a61217..6fb9279a5b 100644
--- a/src/main/groovy/groovy/grape/GrapeIvy.groovy
+++ b/src/main/groovy/groovy/grape/GrapeIvy.groovy
@@ -210,7 +210,7 @@ class GrapeIvy implements GrapeEngine {
dep.each { k, v ->
if (v instanceof CharSequence) {
if (k.toString().contains('v')) { // revision, version, rev
- if (!(v ==~ '[^\\/:"<>|]*')) {
+ if (!(v ==~ '[^\\\\/:"<>|]*')) {
throw new RuntimeException("Grab: invalid value of '$v' for $k: should not contain any of / \\ : \" < > |")
}
} else {
(groovy) 02/03: use Locale.ROOT in CharSequenceValue.toEnum case fallback
Posted by pa...@apache.org.
This is an automated email from the ASF dual-hosted git repository.
paulk-asert pushed a commit to branch GROOVY_5_0_X
in repository https://gitbox.apache.org/repos/asf/groovy.git
commit 572ddf5f1c4ed554f9bfb949832677042ae06ca0
Author: netliomax25-code <ne...@gmail.com>
AuthorDate: Fri May 29 00:12:22 2026 +0530
use Locale.ROOT in CharSequenceValue.toEnum case fallback
---
.../main/java/org/apache/groovy/json/internal/CharSequenceValue.java | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/subprojects/groovy-json/src/main/java/org/apache/groovy/json/internal/CharSequenceValue.java b/subprojects/groovy-json/src/main/java/org/apache/groovy/json/internal/CharSequenceValue.java
index 49289e7887..1314fdad1d 100644
--- a/subprojects/groovy-json/src/main/java/org/apache/groovy/json/internal/CharSequenceValue.java
+++ b/subprojects/groovy-json/src/main/java/org/apache/groovy/json/internal/CharSequenceValue.java
@@ -24,6 +24,7 @@ import java.math.BigDecimal;
import java.math.BigInteger;
import java.util.Arrays;
import java.util.Date;
+import java.util.Locale;
import java.util.Objects;
import static org.apache.groovy.json.internal.CharScanner.isInteger;
@@ -97,7 +98,7 @@ public class CharSequenceValue implements Value, CharSequence {
try {
return (T) Enum.valueOf(cls, value);
} catch (Exception ex) {
- return (T) Enum.valueOf(cls, value.toUpperCase().replace('-', '_'));
+ return (T) Enum.valueOf(cls, value.toUpperCase(Locale.ROOT).replace('-', '_'));
}
}