From: | Merlin Moncure <mmoncure(at)gmail(dot)com> |
---|---|
To: | Magnus Hagander <magnus(at)hagander(dot)net> |
Cc: | Dave Page <dpage(at)pgadmin(dot)org>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Robert Haas <robertmhaas(at)gmail(dot)com>, Andrew Chernow <ac(at)esilo(dot)com>, Bruce Momjian <bruce(at)momjian(dot)us>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: PQinitSSL broken in some use casesf |
Date: | 2009-02-10 18:06:23 |
Message-ID: | b42b73150902101006n4e20e851l167cbb66b9b96b4c@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Tue, Feb 10, 2009 at 1:02 PM, Magnus Hagander <magnus(at)hagander(dot)net> wrote:
> Merlin Moncure wrote:
>> On Tue, Feb 10, 2009 at 12:03 PM, Dave Page <dpage(at)pgadmin(dot)org> wrote:
>>> On Tue, Feb 10, 2009 at 4:57 PM, Merlin Moncure <mmoncure(at)gmail(dot)com> wrote:
>>>
>>>> PQinitSSL is *broken*. It's always been broken. Since it already
>>>> takes a parameter, I say add a special switch...the backwards
>>>> compatibility danger doesn't seem too bad.
>>> Add a switch to what? I get very nervous for our Windows users when
>>> people start talking about changing the libpq API (for those that
>>> don't know, Windows doesn't have DLL versioning like Unix - so any
>>> non-backwards compatible API change really needs a corresponding
>>> filename change to avoid pain and suffering).
>>
>> PQinitSSL(SSL_ONLY) or something, where the constant is carefully
>> chosen to not be accidentally passed in by older libpq users.
>
> So how are you planinng to deal with it when your application passes
> that to a version of libpq that doesn't support it?
well, either nothing, which is no worse off than we are now, or
backpatch the fix. probably nothing :-)
merlin
From | Date | Subject | |
---|---|---|---|
Next Message | Simon Riggs | 2009-02-10 18:14:29 | Re: HotStandby-Patch and WAL_DEBUG |
Previous Message | Magnus Hagander | 2009-02-10 18:02:51 | Re: PQinitSSL broken in some use casesf |