Hi Claus,

Can't just change logic -- toD to recipientList -- the toD dynamically gets 
http, cxf, sftp, odata...so many uris. 

I will live with limitation for +, rather re-testing every URI. 

Thanks for confirming that + is only which shall not work.

Regards,
Arpit.

-----Original Message-----
From: Claus Ibsen [mailto:claus.ib...@gmail.com] 
Sent: Friday, December 9, 2016 2:41 AM
To: users@camel.apache.org
Subject: Re: toD doesn't support RAW function for passwords (Bug?)

On Fri, Dec 9, 2016 at 11:36 AM, Goyal, Arpit <arpit.go...@sap.com> wrote:
> Ouch.....
>
> And I know the code only does split on '+', but is it possible other special 
> characters also have problem?

No its only the plus sign.

>
> Yeah for the time being we will have to publish limitation for the passwords.
>

Just use recipientList(simple("xxx")) instead of toD



> Regards,
> Arpit.
>
> -----Original Message-----
> From: Claus Ibsen [mailto:claus.ib...@gmail.com]
> Sent: Friday, December 9, 2016 2:21 AM
> To: users@camel.apache.org
> Subject: Re: toD doesn't support RAW function for passwords (Bug?)
>
> recipient list or dont use + in your passwords
>
> On Fri, Dec 9, 2016 at 11:16 AM, Goyal, Arpit <arpit.go...@sap.com> wrote:
>> Hi Claus,
>>
>> What alternative we can use right now? Will any of the following work:
>>
>> Header  - toD("sftp://...password=${header.token}...";)
>> Bean  - toD("sftp://...password=#passBean...";)
>> Properties File  - toD("sftp://...password={{ftp.token}}....";)
>>
>> https://issues.apache.org/jira/browse/CAMEL-10581
>>
>> Regards,
>> Arpit.
>>
>> -----Original Message-----
>> From: Claus Ibsen [mailto:claus.ib...@gmail.com]
>> Sent: Friday, December 9, 2016 12:28 AM
>> To: users@camel.apache.org
>> Subject: Re: toD doesn't support RAW function for passwords (Bug?)
>>
>> Hi
>>
>> Ah yeah that smells like a little bug. You are welcome to log a JIRA
>> http://camel.apache.org/support.html
>>
>> On Fri, Dec 9, 2016 at 3:50 AM, Goyal, Arpit <arpit.go...@sap.com> wrote:
>>> Hello Camel users,
>>>
>>> On my 5-10mins analysis - looks like a problem/bug - please can you clarify?
>>>
>>> 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 assume either it 
>>> doesn't support RAW or this was missed out and is a bug.
>>>
>>> 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.
>>>
>>> Ref: 
>>> http://camel.apache.org/how-do-i-configure-password-options-on-camel-endpoints-without-the-value-being-encoded.html
>>>
>>
>>
>>
>> --
>> Claus Ibsen
>> -----------------
>> http://davsclaus.com @davsclaus
>> Camel in Action 2: https://www.manning.com/ibsen2
>
>
>
> --
> Claus Ibsen
> -----------------
> http://davsclaus.com @davsclaus
> Camel in Action 2: https://www.manning.com/ibsen2



-- 
Claus Ibsen
-----------------
http://davsclaus.com @davsclaus
Camel in Action 2: https://www.manning.com/ibsen2

Reply via email to