Closed rwhitcroft closed 2 years ago
I'll be happy to review this and work with you on getting it merged in 😄
Just to let you know timeline I'm teaching this week, so I'll review and provide any feedback monday or tuesday of next week.
Thank you for the review and suggestions! I think I got them all fixed.
Thanks for making those changes! merged
Test if a remote port is open, similar to
nc -z
.Issues:
struct addrinfo
directly inbofdefs.h
, but not sure this is allowed. I did this because including<ws2tcpip.h>
results in warnings from other bofs that include<windows.h>
beforebofdefs.h
.SA.cna
part but tried to copy the other ones.I realize this is not a groundbreaking addition to the repo, so no big deal if it's too much of a hassle to add. I had a blast learning BOF stuff either way.