| From: | Thomas Munro <thomas(dot)munro(at)gmail(dot)com> |
|---|---|
| To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
| Cc: | Jacob Champion <jacob(dot)champion(at)enterprisedb(dot)com>, Nazir Bilal Yavuz <byavuz81(at)gmail(dot)com>, Andres Freund <andres(at)anarazel(dot)de>, Daniel Gustafsson <daniel(at)yesql(dot)se>, Peter Eisentraut <peter(at)eisentraut(dot)org>, Antonin Houska <ah(at)cybertec(dot)at>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: [PoC] Federated Authn/z with OAUTHBEARER |
| Date: | 2025-03-19 04:34:18 |
| Message-ID: | CA+hUKGKCYRS34L+BUy5qhMF55Zg-oCMxDhnSgqm0jWkP+6jYBA@mail.gmail.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Wed, Mar 19, 2025 at 5:17 PM Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)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...
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Michael Paquier | 2025-03-19 04:49:23 | Re: Add Pipelining support in psql |
| Previous Message | Tom Lane | 2025-03-19 04:17:15 | Re: [PoC] Federated Authn/z with OAUTHBEARER |