| 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: | Whole Thread | Raw Message | 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 |