Re: [PATCH] Add an ldapoption to disable chasing LDAP referrals
От | James Sewell |
---|---|
Тема | Re: [PATCH] Add an ldapoption to disable chasing LDAP referrals |
Дата | |
Msg-id | CANkGpBu-Bj4_tRSr2jPvQOG9vfzAqUnKQNus2mct5i_7H+SkdA@mail.gmail.com обсуждение исходный текст |
Ответ на | Re: [PATCH] Add an ldapoption to disable chasing LDAP referrals (Peter Eisentraut <peter_e@gmx.net>) |
Ответы |
Re: [PATCH] Add an ldapoption to disable chasing LDAP
referrals
|
Список | pgsql-hackers |
Hey All,
I had missed these emails, sorry.
The search+bind mode issue is one of documentation location, I have fixed it by moving the section to the applied to both list. As the patch is to do with post-auth response this is correct.
As far as the issue when something other than 0 or 1 is set I am happy throw an error (although this doesn't seem to be how option such as LDAPTLS work: 1 if 1 else 0). I assume I would use the ereport() function to do this (using the second example from this page http://www.postgresql.org/docs/9.2/static/error-message-reporting.html)?
Cheers,
James
James Sewell,
PostgreSQL Team Lead / Solutions Architect
______________________________________

On Thu, Sep 19, 2013 at 12:56 AM, Peter Eisentraut <peter_e@gmx.net> wrote:
On 7/8/13 9:33 PM, James Sewell wrote:When ldapreferrals is set to something other than "0" or "1" exactly, it
> New patch attached. I've moved from using a boolean to an enum trivalue.
just ignores the option. That's not good, I think. It should probably
be an error.
The contents of this email are confidential and may be subject to legal or professional privilege and copyright. No representation is made that this email is free of viruses or other defects. If you have received this communication in error, you may not copy or distribute any part of it or otherwise disclose its contents to anyone. Please advise the sender of your incorrect receipt of this correspondence.
В списке pgsql-hackers по дате отправления: