Обсуждение: initdb -A ident, with params

Поиск
Список
Период
Сортировка

initdb -A ident, with params

От
Magnus Hagander
Дата:
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/


Re: initdb -A ident, with params

От
Devrim GÜNDÜZ
Дата:
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

Re: initdb -A ident, with params

От
Magnus Hagander
Дата:
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/