Thanks Akshay. Dhiraj, can you review please? I'm a little busy right now.

Thanks.


On Thu, May 15, 2014 at 7:39 AM, Akshay Joshi <[email protected]
> wrote:

> Hi Dave
>
> I have fixed the escaping issue and tested it. It works fine for me.
> Attached is the patch file, can you please review it.
> If code looks good to you, can you please commit the code.
>
>
> On Thu, May 8, 2014 at 2:34 PM, Akshay Joshi <
> [email protected]> wrote:
>
>> Sure.
>>
>>
>> On Thu, May 8, 2014 at 1:37 PM, Dave Page <[email protected]> wrote:
>>
>>> Akshay, can you look into the quoting problem please.
>>>
>>> On Thu, May 8, 2014 at 1:07 AM, Stephen Frost <[email protected]>
>>> wrote:
>>> > * Heikki Linnakangas ([email protected]) wrote:
>>> >> (forwarding to pgadmin-hackers)
>>> >
>>> > Ah.
>>> >
>>> >> On 05/07/2014 06:44 PM, Stephen Frost wrote:
>>> >> >* [email protected] ([email protected]) wrote:
>>> >> >>but when the credential contains the delimiter (colon) it fails to
>>> be
>>> >> >>read back out and app responds with "invalid credentials".
>>> >> >>
>>> >> >>x.x.x.x:5432:*:username:password:with:colons
>>> >> >
>>> >> >Per the fine documentation, you need to escape any such usage with a
>>> >> >backslash.  Please review:
>>> >>
>>> >> Stephen, you missed the context. pgadmin3 saves .pgpass, when you
>>> >> check the "store password" checkbox in the connection dialog. And
>>> >> apparantly pgadmin3 doesn't do that escaping properly.
>>> >
>>> > Wow, that's pretty rough.  Hopefully they'll be able to fix it soon. :)
>>> >
>>> >         Thanks,
>>> >
>>> >                 Stephen
>>>
>>>
>>>
>>> --
>>> Dave Page
>>> Blog: http://pgsnake.blogspot.com
>>> Twitter: @pgsnake
>>>
>>> EnterpriseDB UK: http://www.enterprisedb.com
>>> The Enterprise PostgreSQL Company
>>>
>>
>>
>>
>> --
>> *Akshay Joshi*
>> *Principal Software Engineer *
>>
>>
>>
>> *Phone: +91 20-3058-9517 <%2B91%2020-3058-9517> Mobile: +91 976-788-8246
>> <%2B91%20976-788-8246>*
>>
>
>
>
> --
> *Akshay Joshi*
> *Principal Software Engineer *
>
>
>
> *Phone: +91 20-3058-9517 <%2B91%2020-3058-9517> Mobile: +91 976-788-8246
> <%2B91%20976-788-8246>*
>



-- 
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

Attachment: Fixed_unescaped_password.patch
Description: Binary data

-- 
Sent via pgadmin-hackers mailing list ([email protected])
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgadmin-hackers

Reply via email to