[
https://issues.apache.org/jira/browse/PROTON-725?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
German Shepherd closed PROTON-725.
----------------------------------
Resolution: Not a Problem
Fix Version/s: 0.8
Let me provide an update.
MS Azure does actually accept the URL-escaped strings as a 'password', so there
is no need to un-escape the values in ProtonC, after the URL is parsed.
It was an issue of the Provisioning, which provided the plain Base64 text for
'password', without escaping it first.
First, I was afraid that MS Azure won't accept the escaped 'password'.
After testing such a setup, I found out that the MS Azure does accept it and
the SASL goes through OK in such case.
Thanks for a discussion, I'm closing the ticket (no need for any ProtonC code
modification due to this ticket).
> pni_parse_url does not properly handle Base64 in URL
> ----------------------------------------------------
>
> Key: PROTON-725
> URL: https://issues.apache.org/jira/browse/PROTON-725
> Project: Qpid Proton
> Issue Type: Bug
> Components: proton-c
> Affects Versions: 0.8
> Reporter: German Shepherd
> Priority: Minor
> Fix For: 0.8
>
>
> Our provisioning system provides us with a password in Base64 format, which
> can actually contain a slash character(s): '/'
> This breaks the ability of {{pni_parse_url()}} to handle such URLs.
> Example of URL string on which the parsing fails:
> {{amqps://0ABCDEF1:/ExlxtfN+UxfExRxuwxIeFxeFzXLxxg+fxPx3x8xxU=@myself.servicebus.windows.net/out/txxxx/Subscriptions/syyyy}}
> Instead of filling up the "user" and "password" variables, it puts the actual
> contents to "host" and "port".
> With the actual code it is a correct behavior - and *it is* also mentioned in
> the comment right in front of the pni_parse_url() function.
> Yet I would need an update to the functionality, to handle the Base64 in
> password properly (anyone on Azure platform might run to this issue, soon or
> later).
> Note. I'm running the SVN rev 1633464 (latest trunk of 10/23/2014) version of
> ProtonC (C / Linux).
> Thanks for looking at this.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
