[CAMEL-10581] toD (ToDynamicDefinition) does not honor RAW( ) contract - 'removes + from password'

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.16.3
• Fix Version/s: 2.17.5, 2.18.2, 2.19.0
• Component/s: camel-core
• Labels: password raw sftp toD
• Estimated Complexity: Unknown
• Assignee: Claus Ibsen
• Reporter: Arpit Goyal
• Created: 09/Dec/16 10:12
• Updated: 05/Jan/17 00:31
• Resolved: 14/Dec/16 10:38

Description

We have SFTP passwords which have special characters and hence we tried to use the RAW function. On testing we saw that the RAW doesn't work and it still fails.

On debugging we realize that we are using toD("sftp://....") and not to("sftp://..."). As toD has been introduced later, I guess support for RAW was missed
Camel Version: 2.16.3

URL: sftp://10.14.112.183:8686/customer?username=tester&password=RAW(abXNd+4nfzS)&stepwise=false&disconnect=true&reconnectDelay=5000&autoCreate=false&fileExist=Override&connectTimeout=300000

Code Fails at - org.apache.camel.model.ToDynamicDefinition.createExpression(RouteContext) - line 86 or 117 (depending on your viewpoint).

• Line 86: String[] parts = uri.split("
  +");
• Line 117: We concat expressions, without realizing that + was within the RAW function.

Regards,
Arpit.