Horton

Re: Raising the SCRAM iteration count

Michael Paquier <michael@paquier.xyz>

View thread View on postgr.es
From: Michael Paquier <michael@paquier.xyz>
To: Daniel Gustafsson <daniel@yesql.se>
Cc: "Gregory Stark (as CFM)" <stark.cfm@gmail.com>, "Jonathan S. Katz" <jkatz@postgresql.org>, Andres Freund <andres@anarazel.de>, PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
Date: 2023-03-23T23:33:49Z
Lists: pgsql-hackers 
On Thu, Mar 23, 2023 at 10:46:56PM +0100, Daniel Gustafsson wrote:
> I'm fairly convinced it's a timeout in the interactive psql session.  Given how
> ugly the use of that is I'm sort of waiting for Andres' refactoring patch [0] to
> commit this such that I can rewrite the test in a saner and more robust way.

FWIW, I'd be OK here even if you don't have a test for libpq in the
first change as what you have sent is already testing for the core
machinery in scram-common.c.  You could always add one later.
--
Michael

Commits

  1. Test SCRAM iteration changes with psql \password

  2. Make SCRAM iteration count configurable