Re: dblink: Add SCRAM pass-through authentication

On 21.03.25 19:24, Matheus Alcantara wrote:
> On Fri, Mar 21, 2025 at 1:28 PM Jacob Champion
> <jacob.champion@enterprisedb.com> wrote:
>>
>> Great, thank you! Looking over v10, I think I've run out of feedback
>> at this point. Marked Ready for Committer.
> 
> Thanks for all the effort reviewing this patch!

I have committed the 0003 patch (the postgres_fdw bug fix).

The dblink feature patch (0002) looks good to me.

I'm a bit confused about the refactoring patch 0001.  There are some 
details there that don't seem right.  For example, you write that the 
pfree(rconn) calls are no longer necessary, but AFAICT, it would still 
be needed in dblink_get_conn().  Also, there appear to be some possible 
behavior changes, or at least it's not fully explained, like 
connect_pg_server() doing foreign-server name resolution, which 
dblink_get_conn() did not do before.

But it's actually not clear to me how the refactoring in 0001 
contributes to making the patch 0002 better, since patch 0002 barely 
touches the code touched by 0001.

How would patch 0002 look without 0001 before it?  Which code would need 
to be duplicated or what other awkward changes are you trying to avoid?