Re: Indent authentication overloading
Magnus Hagander <magnus@hagander.net>
From: Magnus Hagander <magnus@hagander.net>
To: Tom Lane <tgl@sss.pgh.pa.us>
Cc: Robert Haas <robertmhaas@gmail.com>, Peter Eisentraut <peter_e@gmx.net>, Bruce Momjian <bruce@momjian.us>, PostgreSQL-development <pgsql-hackers@postgresql.org>
Date: 2011-03-19T13:14:18Z
Lists: pgsql-hackers
Attachments
- ident2peer.patch (text/x-patch) patch
On Mon, Mar 14, 2011 at 16:26, Magnus Hagander <magnus@hagander.net> wrote: > On Mon, Mar 14, 2011 at 16:17, Tom Lane <tgl@sss.pgh.pa.us> wrote: >> Magnus Hagander <magnus@hagander.net> writes: >>> On Mon, Mar 14, 2011 at 14:43, Robert Haas <robertmhaas@gmail.com> wrote: >>>> Also, the text is not accurate: nothing has been automatically changed >>>> to anything. The pg_hba.conf file is just as it was. You could say >>>> something like "ident" authentication on local socket treated as >>>> "peer", but I think a better idea would be to just remove this message >>>> altogether. >> >>> The idea being to let people know it's been deprecated, nothing else. >>> But sure, we can just remove the message - at elast for now, and maybe >>> add it $n releases down the road when people are expected to have >>> changed over. >> >> I'm with Robert on this one --- the first reaction I had to your >> description of the patch was "why do we need a log message for that?" >> If there were some real reason to push people away from use of the >> non-preferred term, maybe it'd be worth nagging them to change; but >> there isn't. > > Ok. fair enough, I'll take that part out. Here's an updated patch that removes this log message, and adds a few lines to initdb to create a combination of ident/peer rows. And finally, adds docs. > Are people in general in favor of making the change provided I do that, then? Comments? -- Magnus Hagander Me: http://www.hagander.net/ Work: http://www.redpill-linpro.com/