Re: Remove last traces of SCM credential auth from libpq?
Jonathan S. Katz <jkatz@postgresql.org>
From: "Jonathan S. Katz" <jkatz@postgresql.org>
To: Tom Lane <tgl@sss.pgh.pa.us>, Michael Paquier <michael@paquier.xyz>
Cc: Postgres hackers <pgsql-hackers@lists.postgresql.org>
Date: 2023-03-16T17:28:51Z
Lists: pgsql-hackers
On 3/16/23 10:49 AM, Tom Lane wrote: > Michael Paquier <michael@paquier.xyz> writes: >> libpq has kept some code related to the support of authentication with >> SCM credentials for some time now, code dead in the backend since >> 9.1. Wouldn't it be time to let it go and remove this code entirely, >> erroring in libpq if attempting to connect to a server that supports >> that? > > +1. Since that's only used on Unix-domain sockets, it could only be > useful if you were using current libpq while talking to a pre-9.1 > server on the same machine. +1. > Also, in pg_fe_sendauth, couldn't you just let the default: case > handle it instead of adding a bespoke error message? We're not > really expecting that anyone is ever going to hit this, so I'm > not convinced it's worth the translation burden. +1 to this, that was my thought as well. That would let us remove the "AUTH_REQ_SCM_CREDS" constant too. It looks like in the po files there are a bunch of "SCM_CRED authentication method not supported" messages that can also be removed. Thanks, Jonathan
Commits
-
libpq: Remove code for SCM credential authentication
- 98ae2c84a49e 16.0 landed