Re: [PoC] Federated Authn/z with OAUTHBEARER - Mailing list pgsql-hackers

From Andres Freund
Subject Re: [PoC] Federated Authn/z with OAUTHBEARER
Date
Msg-id z2v3lqti3uswlmypmrueqhhikuf6zsyfa4x2rawo4wnwrckihs@22mq45ag7lhw
Whole thread Raw
In response to Re: [PoC] Federated Authn/z with OAUTHBEARER  (Thomas Munro <thomas.munro@gmail.com>)
List pgsql-hackers
Hi,

On 2025-03-19 17:34:18 +1300, Thomas Munro wrote:
> On Wed, Mar 19, 2025 at 5:17 PM Tom Lane <tgl@sss.pgh.pa.us> wrote:
> > fe-auth-oauth-curl.c:1392:49: error: data argument not used by format string [-Werror,-Wformat-extra-args]
> >  1392 |                 actx_error(actx, "deleting kqueue timer: %m", timeout);
> >       |                                  ~~~~~~~~~~~~~~~~~~~~~~~~~~~  ^
> > fe-auth-oauth-curl.c:324:59: note: expanded from macro 'actx_error'
> >   324 |         appendPQExpBuffer(&(ACTX)->errbuf, libpq_gettext(FMT), ##__VA_ARGS__)
> >       |                                                          ~~~     ^
> >
> > "timeout" isn't being used anymore.
> 
> Yeah.  Thanks, fixed.
> 
> Now I'm wondering about teaching CI to fail on compiler warnings, ie
> not just the special warnings task but also in the Mac etc builds.
> The reason it doesn't is because it's sort of annoying to stop the
> main tests because of a format string snafu, but we must be able to
> put a new step at the end after all tests that scans the build logs
> for warning and then raises the alarm...

The best way would probably be to tee the output of the build to a log file
and then have a script at the end to check for errors in that.

Greetings,

Andres Freund



pgsql-hackers by date:

Previous
From: Daniel Gustafsson
Date:
Subject: Re: [PoC] Federated Authn/z with OAUTHBEARER
Next
From: Bruce Momjian
Date:
Subject: Re: PATCH: warn about, and deprecate, clear text passwords