On 2020-05-27 08:00, Michael Paquier wrote: > On Tue, May 26, 2020 at 10:25:25AM +0200, Peter Eisentraut wrote: >> Yeah, I was too enthusiastic about removing that. Here is a better patch. > > + as an MD5 hash. (<literal>on</literal> is also accepted, as an alias > + for <literal>md5</literal>.) The default is > + <literal>scram-sha-256</literal>. > Shouldn't password_encryption = on/true/1/yes be an equivalent of > scram-sha-256 as the default gets changed?
I think these are mostly legacy options anyway, so if we wanted to make a change, we should remove them.
Seems like the better choice yeah. Since we're changing the default anyway, maybe now is the time to do that? Or if not, maybe have it log an explicit deprecation warning when it loads a config with it?