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

fix(PgSQL): Allow to pass IPv6 address in URI notation for postgres #6344

Open
wants to merge 1 commit into
base: 3.8.x
Choose a base branch
from

Conversation

susnux
Copy link

@susnux susnux commented Mar 21, 2024

Q A
Type improvement
Fixed issues

Summary

Allow to pass IPv6 address of postgres server in URI notation ([ff:aa:...]).
When the host is passed in URI format to host parameter of pg_connect it will fail because it then tries to resolve it as when it was a real host name.
Instead just pass the IP address to the hostaddr parameter.

@derrabus
Copy link
Member

derrabus commented Mar 21, 2024

Thank you for your PR.

Please note that we don't merge code changes that are not covered by tests. Is the 3.8.x branch also affected by this bug? If so, please target the 3.8.x branch.

@susnux susnux changed the base branch from 4.0.x to 3.8.x March 21, 2024 21:41
@susnux susnux force-pushed the fix/postgres-ipv6-connection branch 2 times, most recently from 9159548 to 2f9e58c Compare March 21, 2024 23:02
@susnux
Copy link
Author

susnux commented Mar 21, 2024

Please not that we don't merge code changes that are not covered by tests.

I have added a test now.

Is the 3.8.x branch also affected by this bug? If so, please target the 3.8.x branch.

Yes, changed the target.

@susnux susnux force-pushed the fix/postgres-ipv6-connection branch from 2f9e58c to b96a51d Compare March 21, 2024 23:35
@derrabus
Copy link
Member

The Psalm failure is related to your changes.

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
@susnux susnux force-pushed the fix/postgres-ipv6-connection branch from b96a51d to 2360331 Compare March 22, 2024 12:40
@susnux
Copy link
Author

susnux commented Mar 22, 2024

The Psalm failure is related to your changes.

Should be fixed now

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants