On Fri, Mar 17, 2017 at 2:30 AM, Jeff Janes <jeff.ja...@gmail.com> wrote:
> On Thu, Mar 9, 2017 at 4:59 AM, Michael Paquier <michael.paqu...@gmail.com>
> wrote:
>>
>> On Thu, Mar 9, 2017 at 1:17 AM, Joe Conway <m...@joeconway.com> wrote:
>> > On 03/07/2017 08:29 PM, Tom Lane wrote:
>> >> Michael Paquier <michael.paqu...@gmail.com> writes:
>> >>> here is a separate thread dedicated to the following extension for
>> >>> CREATE/ALTER ROLE: PASSWORD ('value' USING 'method').
>> >>
>> >> The parentheses seem weird ... do we really need those?
>> >
>> > +1
>>
>> Seeing 3 opinions in favor of that, let's do so then. I have updated
>> the patch to not use parenthesis.
>
> The regression tests only exercise the CREATE ROLE...USING version, not the
> ALTER ROLE...USING version.Done. > + and <literal>plain</> for an non-hashed password. If the password > + string is already in MD5-hashed or SCRAM-hashed, then it is > + stored hashed as-is. > > In the last line, I think "stored as-is" sounds better. Okay. > Other than that, it looks good to me. Thanks for the review. Attached is an updated patch. -- Michael
0001-Add-clause-PASSWORD-val-USING-protocol-to-CREATE-ALT.patch
Description: Binary data
-- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
