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 [EMAIL PROTECTED] EnterpriseDB http://www.enterprisedb.com + If your life is a hard drive, Christ can be your backup. + ---------------------------(end of broadcast)--------------------------- TIP 9: In versions below 8.0, the planner will ignore your desire to choose an index scan if your joining column's datatypes do not match