From 8b7b0f8e8b2456752680497d28d9ecd83a61fdcb Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?JB=20Onofr=C3=A9?= <jbonofre@apache.org>
Date: Mon, 2 Feb 2026 19:13:43 +0100
Subject: [PATCH] Fix override of numeric value via env variable or system
 property (#2251)

---
 .../core/impl/KarafConfigurationPlugin.java       | 15 ++++++---------
 .../core/impl/KarafConfigurationPluginTest.java   |  5 +++++
 2 files changed, 11 insertions(+), 9 deletions(-)

diff --git a/config/core/src/main/java/org/apache/karaf/config/core/impl/KarafConfigurationPlugin.java b/config/core/src/main/java/org/apache/karaf/config/core/impl/KarafConfigurationPlugin.java
index 56261ca0e41..27099a0fd33 100644
--- a/config/core/src/main/java/org/apache/karaf/config/core/impl/KarafConfigurationPlugin.java
+++ b/config/core/src/main/java/org/apache/karaf/config/core/impl/KarafConfigurationPlugin.java
@@ -48,10 +48,9 @@ public void modifyConfiguration(ServiceReference<?> reference, Dictionary<String
                     properties.put(key, values);
                 } else {
                     value = InterpolationHelper.substVars(value, null,null, convertDictionaryToMap(properties));
-                    try {
-                        int intValue = Integer.parseInt(value);
-                        properties.put(key, intValue);
-                    } catch (NumberFormatException e) {
+                    if (properties.get(key) != null && (properties.get(key) instanceof Number)) {
+                        properties.put(key, Integer.parseInt(value));
+                    } else {
                         properties.put(key, value);
                     }
                 }
@@ -66,14 +65,12 @@ public void modifyConfiguration(ServiceReference<?> reference, Dictionary<String
                     properties.put(key, values);
                 } else {
                     value = InterpolationHelper.substVars(value, null,null, convertDictionaryToMap(properties));
-                    try {
-                        int intValue = Integer.parseInt(value);
-                        properties.put(key, intValue);
-                    } catch (NumberFormatException e) {
+                    if (properties.get(key) != null && (properties.get(key) instanceof Number)) {
+                        properties.put(key, Integer.parseInt(value));
+                    } else {
                         properties.put(key, value);
                     }
                 }
-
             }
         }
     }
diff --git a/config/core/src/test/java/org/apache/karaf/config/core/impl/KarafConfigurationPluginTest.java b/config/core/src/test/java/org/apache/karaf/config/core/impl/KarafConfigurationPluginTest.java
index cd2a138b7bc..3a0c158a57a 100644
--- a/config/core/src/test/java/org/apache/karaf/config/core/impl/KarafConfigurationPluginTest.java
+++ b/config/core/src/test/java/org/apache/karaf/config/core/impl/KarafConfigurationPluginTest.java
@@ -28,14 +28,19 @@ public class KarafConfigurationPluginTest {
     @Test
     public void testSystemProperty() throws Exception {
         System.setProperty("org.apache.karaf.shell.sshPort", "8102");
+        System.setProperty("org.apache.karaf.shell.sshPorts", "[8102,8103]");
         KarafConfigurationPlugin plugin = new KarafConfigurationPlugin();
         Dictionary<String, Object> properties = new Hashtable<>();
         properties.put(Constants.SERVICE_PID, "org.apache.karaf.shell");
         properties.put("foo", "bar");
         properties.put("sshPort", 8101);
+        properties.put("sshPorts", new String[] { "8102" });
         plugin.modifyConfiguration(null, properties);
 
         Assert.assertEquals(8102, properties.get("sshPort"));
+        Assert.assertEquals(2, ((String[])properties.get("sshPorts")).length);
+        Assert.assertEquals("8102", ((String[])properties.get("sshPorts"))[0]);
+        Assert.assertEquals("8103", ((String[])properties.get("sshPorts"))[1]);
         Assert.assertEquals("bar", properties.get("foo"));
     }