Re: [HACKERS] LDAP server docs

2011-11-01 Thread Magnus Hagander
On Mon, Oct 31, 2011 at 20:59, Magnus Hagander mag...@hagander.net wrote:
 On Mon, Oct 31, 2011 at 20:58, Tom Lane t...@sss.pgh.pa.us wrote:
 Magnus Hagander mag...@hagander.net 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


[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
   termliteralldapserver/literal/term
   listitem
para
-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.
/para
   /listitem
  /varlistentry

-- 
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 mag...@hagander.net 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


Re: [HACKERS] LDAP server docs

2011-10-31 Thread Tom Lane
Magnus Hagander mag...@hagander.net 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 Magnus Hagander
On Mon, Oct 31, 2011 at 20:58, Tom Lane t...@sss.pgh.pa.us wrote:
 Magnus Hagander mag...@hagander.net 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