[jira] [Created] (CAMEL-25320) camel-core - the separator of @PropertyInject is used as a regular expression: with | the value is split into single characters, with . there are no values

["shashank (Jira)" <ji...@apache.org>]

shashank created CAMEL-25320:
--------------------------------

             Summary: camel-core - the separator of @PropertyInject is used as a regular expression: with | the value is split into single characters, with . there are no values
                 Key: CAMEL-25320
                 URL: https://issues.apache.org/jira/browse/CAMEL-25320
             Project: Camel
          Issue Type: Bug
          Components: camel-core
            Reporter: shashank


{{CamelPostProcessorHelper}} splits an injected property value into an array, a collection or a map with {{value.split(separator)}} ({{convertArrayUsingSeparator}}, {{convertCollectionUsingSeparator}}, {{convertMapUsingSeparator}}). {{String.split}} takes a regular expression; the annotation documents a separator ("Used for splitting the property value into an array/list of values. For example to use comma to separate the values.").

{code:java}
@PropertyInject(value = "hosts", separator = "|")
public List<String> hosts;        // hosts=serverA|serverB  ->  [s, e, r, v, e, r, A, |, s, ...]
{code}

With {{separator = "|"}} a list or array gets one element per character, a map fails ({{NumberFormatException: For input string: ""}} for {{Map<String, Integer>}}) or gets wrong entries, and with {{separator = "."}} the result is empty.

h3. Reproduction

New {{PropertyInjectSeparatorTest}} (camel-core, {{CamelPostProcessorHelper.getInjectionPropertyValue}} as {{CamelPostProcessorHelperTest}} does): {{|}} for a list and a map and {{.}} for an array fail on main (2 failures, 1 error); {{;}} is the control. Two runs.

h3. Proposed fix

{{Pattern.quote}} the separator in the three methods. The separators of the documentation and tests ({{,}}, {{;}}) split exactly as before; an application that escaped a separator (for example {{separator = "\\\\|"}}) must use the separator itself, and a separator that relied on a regular expression (such as {{"\\\\s+"}}) is no longer supported (the values are trimmed anyway); upgrade guide entry next to the existing @PropertyInject section of CAMEL-25009. With the fix the related tests pass (camel-core {{CamelPostProcessorHelper*}}, {{DefaultCamelBeanPostProcessor*}}, {{CamelBeanPostProcessor*}}, {{PropertyInject*}}: 47; camel-main {{MainIoC*}}: 9).

Found with a Lean 4 model of the split (shared with the camel-aws2-sqs {{batchSeparator}} issue): "the values are the trimmed pieces between the separators" fails on main for {{|}} and {{.}} for every value, and the fix is proved equal to main for every separator character the model treats as plain (only {{|}} and {{.}} are modelled as special; {{$}}, {{^}}, {{?}}, {{*}}, {{+}}, {{(}}, {{[}}, a backslash also fail on main, differently, and are not covered by the proof).

Affected: 4.14.x, 4.18.x and main.

Duplicate check (2026-10-04): JIRA "PropertyInject" with "separator": none on this; CAMEL-25009 (Claus, 2026-09-26) changed the same file without touching the split. GitHub pull requests: none.

_Filed with Claude Code on behalf of allthingssecurity._




--
This message was sent by Atlassian Jira
(v8.20.10#820010)