Re: Patch to support SSL certificate connections

From: Ashesh Vashi <ashesh(dot)vashi(at)enterprisedb(dot)com>
To: Guillaume Lelarge <guillaume(at)lelarge(dot)info>
Cc: Magnus Hagander <magnus(at)hagander(dot)net>, pgadmin-hackers(at)postgresql(dot)org
Subject: Re: Patch to support SSL certificate connections
Date: 2011-01-28 03:58:17
Message-ID: AANLkTi=DkV3cJuDRAu4NGoJ7O6Q8ZjT-TqT9Bw_ThDOD@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Hi Guillaume,

I think - you've used wxStaticBox here.
If I am not wrong, please consider this patch for testing on osx.
I've faced an issue in recent past with the wxStaticBox related to mouse &
keyboard events pass to the components the components under it on osx.

--
Thanks & Regards,

Ashesh Vashi
EnterpriseDB INDIA: Enterprise PostgreSQL Company<http://www.enterprisedb.com>

*http://www.linkedin.com/in/asheshvashi*<http://www.linkedin.com/in/asheshvashi>

On Fri, Jan 28, 2011 at 3:26 AM, Guillaume Lelarge
<guillaume(at)lelarge(dot)info>wrote:

> Le 22/01/2011 01:34, Guillaume Lelarge a écrit :
> > Le 21/01/2011 10:17, Magnus Hagander a écrit :
> >> On Thu, Jan 20, 2011 at 23:08, Guillaume Lelarge <
> guillaume(at)lelarge(dot)info> wrote:
> >>> Le 19/01/2011 00:35, Guillaume Lelarge a écrit :
> >>>> Hi,
> >>>>
> >>>> This patch adds support to specify SSL certificate files and to
> connect
> >>>> according to those.
> >>>>
> >>>> It's pretty straightforward: four file pickers to handle on dlgServer,
> >>>> and changes in the connection code to add those parameters in the
> >>>> connection string.
> >>>>
> >>>
> >>> A screenshot attached so that you can see the UI without building
> pgAdmin :)
> >>>
> >>> Comments?
> >>
> >> I would move root cert up before the others. Also CRL.
> >>
> >> Root cert and CRL are about validating the *server* certificate. Key
> >> and cert file are about SSL authentication of the *client* (thus
> >> *client* certificates). They should definitely be in that order -
> >> perhaps even grouped with a headline?
> >>
> >
> > Done for moving widgets. Not sure grouped widgets could make sense here.
> >
>
> Here is what I finally managed to do. Comments?
>
>
> --
> Guillaume
> http://www.postgresql.fr
> http://dalibo.com
>
>
> --
> Sent via pgadmin-hackers mailing list (pgadmin-hackers(at)postgresql(dot)org)
> To make changes to your subscription:
> http://www.postgresql.org/mailpref/pgadmin-hackers
>
>

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Guillaume Lelarge 2011-01-28 07:25:43 Re: Patch to support SSL certificate connections
Previous Message Dave Page 2011-01-28 01:11:36 Re: Patch to support SSL certificate connections