fix: Fix resolving named hosts.

[matthewwardrop]

Version 0.6 broke our testing pipelines, because it "upgraded" hostnames to IP addresses, but didn't also upgrade the hostname passed in via arguments. This patch resolves this by also resolving hostnames passed in via arguments.

[codeclimate]

Code Climate has analyzed commit 9f7c4b7 and detected 0 issues on this pull request.

View more on Code Climate.

[miketheman]

Thanks @matthewwardrop - I'm not entirely certain how this broke the testing pipeline - could you please add a test that expresses the conditions that you had that this change fixes?

[phil-nelson-bt]

Any chance this is going to get merged? This PR fixes a workaround for the now broken vs code python test runner which uses sockets. with pytest-socket using --disable-socket the test runner fails. The workaround, also not great because you may have false positive tests, but better than not being able to run individual tests in vs code, is to allow localhost. That is what this PR enables

[miketheman]

@phil-nelson-bt The change is waiting for a test, as per my last message.

[matthewwardrop]

Apologies. I'll try to write a test soon

…

On Fri, Dec 15, 2023, 06:45 Mike Fiedler ***@***.***> wrote: @phil-nelson-bt <https://github.com/phil-nelson-bt> The change is waiting for a test, as per my last message. — Reply to this email directly, view it on GitHub <#203 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAA6P3W3QVCTBXWPRRLO7NDYJRO6ZAVCNFSM6AAAAAAWPJHHFSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNJXHE4TKNJTHE> . You are receiving this because you were mentioned.Message ID: ***@***.***>