Re: hba.c:3160:18: warning: comparison of unsigned enum expression

Magnus Hagander <magnus@hagander.net>

From: Magnus Hagander <magnus@hagander.net>
To: Michael Paquier <michael@paquier.xyz>
Cc: Erik Rijkers <er@xs4all.nl>, Jacob Champion <pchampion@vmware.com>, "pgsql-hackers@lists.postgresql.org" <pgsql-hackers@lists.postgresql.org>
Date: 2021-04-07T12:20:25Z
Lists: pgsql-hackers
On Wed, Apr 7, 2021 at 2:17 PM Michael Paquier <michael@paquier.xyz> wrote:
>
> On Wed, Apr 07, 2021 at 02:01:42PM +0200, Magnus Hagander wrote:
> > That's very close to mine (see one email later). Let's bikeshed about
> > the details. I think it's basically the same for current usecases, but
> > that taking the UserAuth as the parameter is cleaner and potentially
> > more useful for the future.
>
> Missed it, sorry about that.  Using UserAuth as argument is fine by
> me.  If you wish to apply that, please feel free.  I am fine to do
> that myself, but that will have to wait until tomorrow my time.

Ok, I'll go ahead and push it. Thanks for confirming the fix!

-- 
 Magnus Hagander
 Me: https://www.hagander.net/
 Work: https://www.redpill-linpro.com/



Commits

  1. Refactor hba_authname

  2. Add some information about authenticated identity via log_connections