Re: Add Option To Check All Addresses For Matching target_session_attr

2025-02-24 Thread Andrew Jackson
The previous patch had a mistake in a reference in the documentation. This should fix it. On Mon, Feb 24, 2025 at 10:07 AM Andrew Jackson wrote: > Looks like this needed another rebase to account for the oauth commit. > Rebase attached. > > On Mon, Feb 24, 2025 at 9:38 AM Andrew Jackson > wrote

Re: Add Option To Check All Addresses For Matching target_session_attr

2025-02-24 Thread Andrew Jackson
Looks like this needed another rebase to account for the oauth commit. Rebase attached. On Mon, Feb 24, 2025 at 9:38 AM Andrew Jackson wrote: > Hi, > > Thank you for the review! > > Review Response > > - Made a first pass at a real commit message > - Fixed the condition on the if statement to us

Re: Add Option To Check All Addresses For Matching target_session_attr

2025-02-24 Thread Andrew Jackson
Hi, Thank you for the review! Review Response - Made a first pass at a real commit message - Fixed the condition on the if statement to use strcmp - Added a test suite in the files `src/interfaces/libpq/t/ 006_target_session_attr_dns.pl` and `src/interfaces/libpq/t/ 007_load_balance_dns_check_al

Re: Add Option To Check All Addresses For Matching target_session_attr

2025-02-16 Thread Andrey Borodin
Hi Andrew! cc Jelte, I suspect he might be interested. > On 20 Nov 2024, at 20:51, Andrew Jackson wrote: > > Would appreciate any feedback on the applicability/relevancy of the goal here > or the implementation. Thank you for raising the issue. Following our discussion in Discord I'm putting

Add Option To Check All Addresses For Matching target_session_attr

2024-11-20 Thread Andrew Jackson
Hi, I was attempting to set up a high availability system using DNS and target_session_attrs. I was using a DNS setup similar to below and was trying to use the connection strings `psql postgresql:// u...@pg.database.com/db_name?target_session=read-write` to have clients dynamically connect to the