Re: Enhancements to passwordcheck

From: Albe Laurenz <laurenz(dot)albe(at)wien(dot)gv(dot)at>
To: "'Bossart, Nathan *EXTERN*'" <bossartn(at)amazon(dot)com>, Euler Taveira <euler(at)timbira(dot)com(dot)br>, Michael Paquier <michael(dot)paquier(at)gmail(dot)com>
Cc: "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Enhancements to passwordcheck
Date: 2017-09-27 09:05:45
Message-ID: A737B7A37273E048B164557ADEF4A58B722200F7@ntex2010i.host.magwien.gv.at
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Nathan Bossart wrote:
>> As was pointed out in the original discussion
>> D960CB61B694CF459DCFB4B0128514C203937F49(at)exadv11(dot)host(dot)magwien(dot)gv(dot)at
>> the weak point of "passwordcheck" is that it does not work very well
>> for encrypted passwords.
>> The only saving grace is that you can at least check against
>> username equals password.
>
> Thanks for linking the original thread. There are a lot of
> interesting points. I wonder if enhanced password checking in core
> or contrib might be received differently with the introduction of
> SCRAM authentication, since the weaknesses of MD5 were often cited.

I had the impression that the reasons why database passwords are
not the best option for high security were:
1) The password hash is stored in the database and can be stolen and
cracked (don't know if dictionary attacks are harder with SCRAM).
2) The password or the password hash are transmitted to the server
when you change the password and may be captured.

>> So I think it is fine to extend "passwordcheck", but we shouldn't
>> take it serious enough to reduce security elsewhere in order to
>> improve the module.
>
> I understand the points made here, but not allowing configurability
> here really hinders the module's ability to enforce much of
> anything.

I agree that it is a good thing to make "passwordcheck" configurable.

Yours,
Laurenz Albe

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Alvaro Herrera 2017-09-27 10:05:59 ALTER enums (was Re: [COMMITTERS] pgsql: doc: first draft of Postgres 10 release notes)
Previous Message Haribabu Kommi 2017-09-27 08:58:22 Re: pg_stat_wal_write statistics view