[
https://issues.apache.org/jira/browse/CAMEL-26465?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Work on CAMEL-26465 started by shashank.
----------------------------------------
> camel-core - Simple: function arguments drop an empty argument, and a comma
> inside a nested function splits an argument
> -----------------------------------------------------------------------------------------------------------------------
>
> Key: CAMEL-26465
> URL: https://issues.apache.org/jira/browse/CAMEL-26465
> Project: Camel
> Issue Type: Bug
> Components: camel-core
> Reporter: shashank
> Assignee: shashank
> Priority: Minor
>
> Found while reviewing CAMEL-26420; the same before it.
> {{SimpleFunctionHelper.splitArguments}} (used by {{replace}}, {{substring}},
> the collection functions and others) and, since CAMEL-26420,
> {{JoinFunctionFactory}} (its own call of the same split, so it can keep the
> separator as written) split with {{StringQuoteHelper.splitSafeQuote(values,
> ',', false, true)}}, which drops an empty argument and does not look at
> nesting:
> * an empty argument is dropped, so the next argument takes its place:
> {{${replace(b,,${header.s})}}} uses the expression as the replacement,
> {{${join(,id=)}}} uses {{id=}} as the separator, {{${join(&,,${body})}}} uses
> the text {{${body}}} as the prefix;
> * a comma inside a nested function splits the argument:
> {{${replace(b,${header.s.substring(1,3)})}}} replaces with {{3)}}},
> {{${join(&,id=,${substring(1,2)})}}} is refused as more than 3 arguments, and
> {{sort}} / {{distinct}} with such an argument fail to parse
> ({{splitSafeQuote}} with {{nested=true}} keeps the nested function together).
> For {{join}} only: {{${join(,,)}}} fails with {{Index 0 out of bounds}}
> instead of the "Valid syntax" message, and {{${join( )}}} gives the default
> separator, as the arguments are checked with {{isNotEmpty}}, which trims.
> *Proposal*
> Split with nesting and keep empty arguments in {{splitArguments}} and in the
> split of {{join}} (best one shared split method in {{SimpleFunctionHelper}},
> with tests per function), and give the "Valid syntax" message of {{join}} for
> a malformed argument list.
> _Filed with Claude Code on behalf of allthingssecurity._
--
This message was sent by Atlassian Jira
(v10.3.26#10030026)
