From: | ivan(dot)kush(at)tantorlabs(dot)com |
---|---|
To: | Jacob Champion <jacob(dot)champion(at)enterprisedb(dot)com>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org>, lev(dot)nikolaev(at)tantorlabs(dot)com |
Subject: | Re: [PoC] Federated Authn/z with OAUTHBEARER |
Date: | 2025-07-10 14:41:07 |
Message-ID: | e0e55734-2b1c-4c7f-81a9-96f9b64c3830@tantorlabs.com |
Views: | Whole Thread | Raw Message | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
I agree with the patch. Works in my OSes
On 7/10/25 2:54 AM, Jacob Champion <jacob(dot)champion(at)enterprisedb(dot)com> wrote:
> On Wed, Jul 9, 2025 at 12:42 PM Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> > Nah, let's keep them. We do document for at least some libraries
> > how to manually specify the include and link options without
> > depending on pkg-config. If someone tries that with libcurl,
> > it'd be good to have sanity checks on the results.
>
> Sounds good, thanks for the review!
>
> On Wed, Jul 9, 2025 at 11:39 AM Jacob Champion
> <jacob(dot)champion(at)enterprisedb(dot)com> wrote:
> > Here is a draft patch for Ivan's reported issue; I still need to put
> > it through its paces with some more unusual setups, but I want to get
> > cfbot on it.
>
> On HEAD, Rocky 9 fails to build with a custom Curl PKG_CONFIG_PATH and
> no libcurl-devel installed. With this patch, that build now succeeds,
> and it still succeeds after libcurl-devel is reinstalled, with the
> compiler tests continuing to use the custom libcurl and not the
> system's.
>
> So I'll give Ivan a little time in case he'd like to test/review
> again, but otherwise I plan to push it this week.
>
> Thanks,
> --Jacob
>
From | Date | Subject | |
---|---|---|---|
Next Message | Amit Langote | 2025-07-10 14:54:41 | Re: Problem with transition tables on partitioned tables with foreign-table partitions |
Previous Message | Robert Haas | 2025-07-10 14:35:48 | Re: Fix some inconsistencies with open-coded visibilitymap_set() callers |