Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DNS lookups with connection pooling #731

Open
chrooti opened this issue Nov 22, 2023 · 0 comments
Open

DNS lookups with connection pooling #731

chrooti opened this issue Nov 22, 2023 · 0 comments
Assignees

Comments

@chrooti
Copy link

chrooti commented Nov 22, 2023

When a connection is reused due to pooling it will still do a DNS lookup via the is_ipv6 function while calling connect_options inside the checkout. I don't know if this is intentional or not.

A fix would probably involve having the output of connect_options both include inet in the ipv4 case and modify the cached client here, I can try my hand at it if that's confirmed.

Until this is fixed/if this is not a bug, a workaround is adding to the options a field connect_options: [:inet].

@benoitc benoitc self-assigned this Nov 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants