Re: Patch to support SSL certificate connections
От | Dave Page |
---|---|
Тема | Re: Patch to support SSL certificate connections |
Дата | |
Msg-id | AANLkTikzpjgzKe=ONA+YgRd=Dp47NZ5Hc+SmAvHzr6YT@mail.gmail.com обсуждение исходный текст |
Ответ на | Re: Patch to support SSL certificate connections (Guillaume Lelarge <guillaume@lelarge.info>) |
Ответы |
Re: Patch to support SSL certificate connections
(Guillaume Lelarge <guillaume@lelarge.info>)
|
Список | pgadmin-hackers |
That's better. Fyi, the other design did look OK - i just think we need to plan such changes properly and implement in batches rather than piecemeal. On 1/28/11, Guillaume Lelarge <guillaume@lelarge.info> wrote: > Le 28/01/2011 09:37, Guillaume Lelarge a écrit : >> Le 28/01/2011 09:21, Dave Page a écrit : >>> On Fri, Jan 28, 2011 at 7:25 AM, Guillaume Lelarge >>> <guillaume@lelarge.info> wrote: >>>> Le 28/01/2011 02:11, Dave Page a écrit : >>>>> On Thu, Jan 27, 2011 at 9:56 PM, Guillaume Lelarge >>>>> <guillaume@lelarge.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@lelarge.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? >>>>> >>>>> Why the change from the standard layout used everywhere else? >>>>> >>>> >>>> Because Magnus asked for that kind of design? (grouped with a headline) >>> >>> He suggested it, but that doesn't mean it has to be done. If we're >>> going to break the property dialog design guidelines and start laying >>> things out in new formats we need to be *really* careful to avoid >>> getting in an inconsistent mess. >>> >> >> Yeah, I know. And "grouped with a headline" doesn't mean we have to use >> a wxStaticBoxSizer. Anyway, reading your comments and Ashesh's comments, >> I will drop the wxStaticBoxSizer and everyone will be happy: >> >> * no strange issue on MacOSX for Ashesh >> * no weird design for you >> * but still first server certificate and then client certificate for >> Magnus. >> >> Will try to work on this today. >> > > What about this one? > > > -- > Guillaume > http://www.postgresql.fr > http://dalibo.com > -- Dave Page Blog: http://pgsnake.blogspot.com Twitter: @pgsnake EnterpriseDB UK: http://www.enterprisedb.com The Enterprise PostgreSQL Company
В списке pgadmin-hackers по дате отправления: