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 Mobile: +91 976-788-8246* > -- *Akshay Joshi* *Principal Software Engineer * *Phone: +91 20-3058-9517Mobile: +91 976-788-8246*
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
