| From: | "Albe Laurenz" <all(at)adv(dot)magwien(dot)gv(dot)at> |
|---|---|
| To: | "Bruce Momjian *EXTERN*" <bruce(at)momjian(dot)us> |
| Cc: | <pgsql-patches(at)postgresql(dot)org> |
| Subject: | Re: LDAP lookup of connection parameters |
| Date: | 2006-07-27 12:57:39 |
| Message-ID: | 52EF20B2E3209443BC37736D00C3C13809A923B9@EXADV1.host.magwien.gv.at |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-patches |
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
| Attachment | Content-Type | Size |
|---|---|---|
| ldap_v2.patch | application/octet-stream | 16.1 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Simon Riggs | 2006-07-27 13:08:52 | WAL file rotation/ XLog Switch |
| Previous Message | Tom Lane | 2006-07-27 12:30:21 | Re: extension for sql update |