Re: [HACKERS] LDAP server docs

2011-11-01 Thread Magnus Hagander
On Mon, Oct 31, 2011 at 20:59, Magnus Hagander  wrote:
> On Mon, Oct 31, 2011 at 20:58, Tom Lane  wrote:
>> Magnus Hagander  writes:
>>> So once again I forgot about the fact that you can specify multiple
>>> LDAP server in our ldapserver parameter (because both openldap and
>>> winldap accept a space separated list).
>>
>>> Any objections to just applying the attached docs patch?
>>
>> "space-separated list" is more in keeping with our usual terminology,
>> I think, but otherwise please do.
>
> FWIW, the use of the word blank was just because I copied it off the
> ldap manpage. I agree space is better :-)

Updated and applied.

-- 
 Magnus Hagander
 Me: http://www.hagander.net/
 Work: http://www.redpill-linpro.com/

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers


Re: [HACKERS] LDAP server docs

2011-10-31 Thread Magnus Hagander
On Mon, Oct 31, 2011 at 20:58, Tom Lane  wrote:
> Magnus Hagander  writes:
>> So once again I forgot about the fact that you can specify multiple
>> LDAP server in our ldapserver parameter (because both openldap and
>> winldap accept a space separated list).
>
>> Any objections to just applying the attached docs patch?
>
> "space-separated list" is more in keeping with our usual terminology,
> I think, but otherwise please do.

FWIW, the use of the word blank was just because I copied it off the
ldap manpage. I agree space is better :-)


-- 
 Magnus Hagander
 Me: http://www.hagander.net/
 Work: http://www.redpill-linpro.com/

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers


Re: [HACKERS] LDAP server docs

2011-10-31 Thread Tom Lane
Magnus Hagander  writes:
> So once again I forgot about the fact that you can specify multiple
> LDAP server in our ldapserver parameter (because both openldap and
> winldap accept a space separated list).

> Any objections to just applying the attached docs patch?

"space-separated list" is more in keeping with our usual terminology,
I think, but otherwise please do.

regards, tom lane

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers


Re: [HACKERS] LDAP server docs

2011-10-31 Thread Robert Haas
On Mon, Oct 31, 2011 at 3:11 PM, Magnus Hagander  wrote:
> So once again I forgot about the fact that you can specify multiple
> LDAP server in our ldapserver parameter (because both openldap and
> winldap accept a space separated list).
>
> Any objections to just applying the attached docs patch?

I think the term "space" might be more clear than "blank" here.  Perhaps:

Names or IP addresses of LDAP servers to connect to.  Multiple servers
may be specified, separated by spaces.

-- 
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers


[HACKERS] LDAP server docs

2011-10-31 Thread Magnus Hagander
So once again I forgot about the fact that you can specify multiple
LDAP server in our ldapserver parameter (because both openldap and
winldap accept a space separated list).

Any objections to just applying the attached docs patch?

-- 
 Magnus Hagander
 Me: http://www.hagander.net/
 Work: http://www.redpill-linpro.com/
diff --git a/doc/src/sgml/client-auth.sgml b/doc/src/sgml/client-auth.sgml
index 5d543cb..3734586 100644
--- a/doc/src/sgml/client-auth.sgml
+++ b/doc/src/sgml/client-auth.sgml
@@ -1387,7 +1387,8 @@ omicron bryanh  guest1
   ldapserver
   

-Name or IP of LDAP server to connect to.
+Name or IP of LDAP server to connect to. You may specify a
+blank separated list of hostnames to try to connect to.

   
  

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers