(commons-beanutils) branch master updated: Reject out-of-range color components in ColorConverter (#420).

[ggregory]

This is an automated email from the ASF dual-hosted git repository.

garydgregory pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-beanutils.git


The following commit(s) were added to refs/heads/master by this push:
     new c1a2b191 Reject out-of-range color components in ColorConverter (#420).
c1a2b191 is described below

commit c1a2b1910c762acc3fe138294ba02a9ce79452f8
Author: Gary Gregory <garydgreg...@gmail.com>
AuthorDate: Fri Jul 17 07:22:33 2026 -0700

    Reject out-of-range color components in ColorConverter (#420).
    
    Add assertions.
---
 src/changes/changes.xml                                                 | 1 +
 .../org/apache/commons/beanutils2/converters/ColorConverterTest.java    | 2 ++
 2 files changed, 3 insertions(+)

diff --git a/src/changes/changes.xml b/src/changes/changes.xml
index 339068db..2f56e39b 100644
--- a/src/changes/changes.xml
+++ b/src/changes/changes.xml
@@ -46,6 +46,7 @@
       <action type="fix" dev="ggregory" due-to="Naveed Khan, Gary 
Gregory">Make MethodUtils CACHE_ENABLED volatile (#414).</action>
       <action type="fix" dev="ggregory" due-to="Naveed Khan, Gary 
Gregory">Case-fold column and bean property names with Locale.ROOT 
(#415).</action>
       <action type="fix" dev="ggregory" due-to="Naveed Khan, Gary 
Gregory">Make WrapDynaClass instance cache thread-safe (#418).</action>
+      <action type="fix" dev="ggregory" due-to="Naveed Khan, Gary 
Gregory">Reject out-of-range color components in ColorConverter (#420).</action>
       <!-- ADD -->
       <action type="add" dev="ggregory" due-to="Gary Gregory">Use varargs for 
org.apache.commons.beanutils2.MethodUtils.getAccessibleMethod(Class, String, 
Class...), and remove last argument variants using Class[] and Class.</action>
       <action type="add" dev="ggregory" due-to="SethFalco, Melloware, Gary 
Gregory">Add Instant converter and improve enum converter #49.</action>
diff --git 
a/src/test/java/org/apache/commons/beanutils2/converters/ColorConverterTest.java
 
b/src/test/java/org/apache/commons/beanutils2/converters/ColorConverterTest.java
index 5261bce3..0bba5c84 100644
--- 
a/src/test/java/org/apache/commons/beanutils2/converters/ColorConverterTest.java
+++ 
b/src/test/java/org/apache/commons/beanutils2/converters/ColorConverterTest.java
@@ -110,6 +110,8 @@ class ColorConverterTest {
         assertThrows(ConversionException.class, () -> 
converter.convert(Color.class, "1234,5,6"));
         assertThrows(ConversionException.class, () -> 
converter.convert(Color.class, "1000,0,0"));
         assertThrows(ConversionException.class, () -> 
converter.convert(Color.class, "2550,0,0"));
+        assertThrows(ConversionException.class, () -> 
converter.convert(Color.class, "0,2550,0"));
+        assertThrows(ConversionException.class, () -> 
converter.convert(Color.class, "0,0,2550"));
     }
 
     @Test