Added open? method #18

Closed
wants to merge 4 commits into
from

Conversation

Projects
None yet
4 participants
@thesp0nge

Hi there, I added an open?(addr, port) method to check if a socket can successfully opened to that pair.

@t3hk0d3

This comment has been minimized.

Show comment
Hide comment
@t3hk0d3

t3hk0d3 Sep 28, 2012

Test is environment dependent. This is bad :(

Use mocking

Test is environment dependent. This is bad :(

Use mocking

This comment has been minimized.

Show comment
Hide comment

^ 👍

@francisluong

This comment has been minimized.

Show comment
Hide comment
@francisluong

francisluong Sep 10, 2016

Member

Closing as out of scope for ipaddress. (and also the PR is pretty old)

Member

francisluong commented Sep 10, 2016

Closing as out of scope for ipaddress. (and also the PR is pretty old)

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