Re: Document use of ldapurl with LDAP simple bind

From: Jacob Champion <jacob(dot)champion(at)enterprisedb(dot)com>
To: Peter Eisentraut <peter(at)eisentraut(dot)org>
Cc: PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Document use of ldapurl with LDAP simple bind
Date: 2024-07-08 21:27:12
Message-ID: CAOYmi+m1xrWwGyOUPopQvYgOyPJa+TYAq3uZ09MmgkZq6RmCKA@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Fri, Jun 28, 2024 at 12:11 AM Peter Eisentraut <peter(at)eisentraut(dot)org> wrote:
> This appears to imply that specifying ldapurl is only applicable for
> search+bind. Maybe that whole message should be simplified to something
> like
>
> "configuration mixes arguments for simple bind and search+bind"
>
> (The old wording also ignores that the error might arise via "ldapsuffix".)

I kept the imperative "cannot" and tried to match the terminology with
our documentation at [1]:

cannot mix options for simple bind and search+bind modes

WDYT?

--Jacob

[1] https://www.postgresql.org/docs/17/auth-ldap.html

Attachment Content-Type Size
v2-0001-docs-explain-how-to-use-ldapurl-with-simple-bind.patch application/octet-stream 2.7 KB
v2-0002-ldap-test-ldapurl-with-simple-bind.patch application/octet-stream 1.4 KB
v2-0003-hba-improve-error-when-mixing-LDAP-bind-modes.patch application/octet-stream 1.3 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Andres Freund 2024-07-08 21:44:21 Re: tests fail on windows with default git settings
Previous Message Andres Freund 2024-07-08 21:26:40 Re: 010_pg_basebackup.pl vs multiple filesystems