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 a7apd2ywdtigk56cxtvmg4xhxktkqhhubgzljwpmmf3dlpztkp@mr7l2vx6i222
Whole thread Raw
In response to Re: [PoC] Federated Authn/z with OAUTHBEARER  (Bruce Momjian <bruce@momjian.us>)
Responses Re: [PoC] Federated Authn/z with OAUTHBEARER
List pgsql-hackers
Hi,

On 2025-04-08 11:13:51 -0400, Bruce Momjian wrote:
> On Tue, Apr  8, 2025 at 08:04:22AM -0700, Jacob Champion wrote:
> > On Tue, Apr 8, 2025 at 7:32 AM Bruce Momjian <bruce@momjian.us> wrote:
> > > Uh, where are we on the inclusion of curl in our build?  Maybe it was
> > > explained but I have not seen it.
> > 
> > The above is discussing a patch to split this into its own loadable
> > module. Andres and Christoph's feedback has been shaping where we put
> > that module, exactly.
> 
> Uh, I was afraid that was the case, which is why I asked.  We have just
> hit feature freeze, so it is not good that we are still "shaping" the
> patch.  Should we consider reverting it?  It is true we still "adjust"
> patches after feature freeze.

You brought the dependency concern up well after the feature was merged, after
it had been in development for a *long* time. It wasn't a secret that it had a
dependency on curl.  I don't think it's fair to penalize a feature's authors
to not finish implementing a complicated and completely new requirement within
17 days.

Greetings,

Andres Freund



pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: Feature freeze
Next
From: Nathan Bossart
Date:
Subject: Re: Feature freeze