From: | Daniel Gustafsson <daniel(at)yesql(dot)se> |
---|---|
To: | Michael Paquier <michael(at)paquier(dot)xyz> |
Cc: | "Jonathan S(dot) Katz" <jkatz(at)postgresql(dot)org>, Andres Freund <andres(at)anarazel(dot)de>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
Subject: | Re: Raising the SCRAM iteration count |
Date: | 2023-03-09 10:01:57 |
Message-ID: | 2822C852-9890-4CC1-870B-C20AECD11045@yesql.se |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
> On 9 Mar 2023, at 08:09, Michael Paquier <michael(at)paquier(dot)xyz> wrote:
>
> On Wed, Mar 08, 2023 at 05:21:20PM +0900, Michael Paquier wrote:
>> On Wed, Mar 08, 2023 at 09:07:36AM +0100, Daniel Gustafsson wrote:
>>> AFAIK a TAP test with psql_interactive is the only way to do this so that's
>>> what I've implemented.
>
> I cannot think of a better idea than what you have here, so I am
> marking this patch as ready for committer.
Thanks for review!
> I am wondering how stable a logic based on a timer of 5s would be..
Actually that was a bug, it should be using the default timeout and restarting
for each operation to ensure that even overloaded hosts wont time out unless
something is actually broken/incorrect. I've fixed that in the attached rev
and also renamed the password in the regress test from "raisediterationcount"
as it's now lowering the count in the test.
Unless there objections to this version I plan to commit that during this CF.
--
Daniel Gustafsson
Attachment | Content-Type | Size |
---|---|---|
v8-0001-Make-SCRAM-iteration-count-configurable.patch | application/octet-stream | 17.0 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Peter Eisentraut | 2023-03-09 10:21:25 | Re: Add standard collation UNICODE |
Previous Message | houzj.fnst@fujitsu.com | 2023-03-09 09:58:52 | RE: Support logical replication of DDLs |