From: | Bruce Momjian <bruce(at)momjian(dot)us> |
---|---|
To: | Albe Laurenz <all(at)adv(dot)magwien(dot)gv(dot)at> |
Cc: | pgsql-patches(at)postgresql(dot)org |
Subject: | Re: LDAP lookup of connection parameters |
Date: | 2006-07-27 13:20:21 |
Message-ID: | 200607271320.k6RDKLv12022@momjian.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-patches |
Patch applied. Thanks.
---------------------------------------------------------------------------
Albe Laurenz wrote:
> Bruce Momjian wrote:
> >>>> This patch for libpq allows you to enter an LDAP URL in
> >>>> pg_service.conf.
> >>>> The URL will be queried and the resulting string(s) parsed for
> >>>> keyword = value connection options.
> >>>
> >>> I have heavily modified your patch to be clearer. Please
> >>> review the
> >>> attached version and test it to make sure it still works
> >>> properly.
> >
> > Thanks for the review. Updated patch attached. Is that OK?
>
> Mostly yes, and I must admit that the code has become more
> readable. There were two syntax errors and some minor odds
> and ends (partly my fault).
>
> I have slightly changed your patch and submit it again
> (diff the patches to see).
>
> Thanks for the effort,
> Laurenz Albe
>
Content-Description: ldap_v2.patch
[ Attachment, skipping... ]
--
Bruce Momjian bruce(at)momjian(dot)us
EnterpriseDB http://www.enterprisedb.com
+ If your life is a hard drive, Christ can be your backup. +
From | Date | Subject | |
---|---|---|---|
Next Message | Bruce Momjian | 2006-07-27 13:28:05 | Re: extension for sql update |
Previous Message | Tom Lane | 2006-07-27 13:17:20 | Re: extension for sql update |