initdb -A ident, with params
In looking at the "peer patch", I notice that initdb still supports
things like "-A ident map=foo", but it *only* supports this for ident,
and nothing else. I'm pretty sure this is a leftover from the "ident
sameuser" days, so that things like the RPM install could set it
properly. Since this is not needed any more, I propose we take that
out and make ident work the same way as any other authentication
methods.
At some point we might want to extend initdb to support any parameters
(with validation, of course), but that's a much bigger thing. Just
removing the (undocumented, btw) special case would make things more
consistent.
Thoughts?
--
Magnus Hagander
Me: http://www.hagander.net/
Work: http://www.redpill-linpro.com/
On Mon, 2011-03-14 at 10:28 +0100, Magnus Hagander wrote:
In looking at the "peer patch", I notice that initdb still supports
things like "-A ident map=foo", but it *only* supports this for ident,
and nothing else. I'm pretty sure this is a leftover from the "ident
sameuser" days, so that things like the RPM install could set it
properly. Since this is not needed any more, I propose we take that
out and make ident work the same way as any other authentication
methods.At some point we might want to extend initdb to support any parameters
(with validation, of course), but that's a much bigger thing. Just
removing the (undocumented, btw) special case would make things more
consistent.Thoughts?
RPMs don't need it anymore, so +1 for removing.
--
Devrim GÜNDÜZ
Principal Systems Engineer @ EnterpriseDB: http://www.enterprisedb.com
PostgreSQL Danışmanı/Consultant, Red Hat Certified Engineer
Community: devrim~PostgreSQL.org, devrim.gunduz~linux.org.tr
http://www.gunduz.org Twitter: http://twitter.com/devrimgunduz
2011/3/14 Devrim GÜNDÜZ <devrim@gunduz.org>:
On Mon, 2011-03-14 at 10:28 +0100, Magnus Hagander wrote:
In looking at the "peer patch", I notice that initdb still supports
things like "-A ident map=foo", but it *only* supports this for ident,
and nothing else. I'm pretty sure this is a leftover from the "ident
sameuser" days, so that things like the RPM install could set it
properly. Since this is not needed any more, I propose we take that
out and make ident work the same way as any other authentication
methods.At some point we might want to extend initdb to support any parameters
(with validation, of course), but that's a much bigger thing. Just
removing the (undocumented, btw) special case would make things more
consistent.Thoughts?
RPMs don't need it anymore, so +1 for removing.
Done, and applied.
--
Magnus Hagander
Me: http://www.hagander.net/
Work: http://www.redpill-linpro.com/