David Campbell

How do you feel about something like this ``` - var finished bool // Read results - for !finished && c.sc.Scan() { - + for { + if ok :=...

`103.57.80.88` is a good example. Even if you use the web client http://whois.cymru.com/ it'll return both ASNs.

> Where exactly are you seeing the delay? The only thing I could think of that could cause that is a deferred statement, but the only thing deferred is a...

> That change looks fine though 👍 Working on a diff

This was the timeout I'm talking about. I'd guess the `net.Conn` read socket isn't closed yet (5 second deadline and 10 second timeout). This change probably requires a little refactoring...

The timeout example I sent was from `go test` against a single IP. I think `Scan` may be a blocking call until a deadline or a timeout happens, which wouldn't...

So I modified the code above to include some debug timing information: ``` + for { + fmt.Println("isp: whoisClient.LookupIPs: entering for loop, waiting to scan") + start := time.Now() +...

I actually tested B: ``` isp: whoisClient.LookupIPs: entering for loop, waiting to scan isp: whoisClient.LookupIPs: scanner is closed taking 5.000203191s ```

Ahh right, the empty struct makes more sense. I was still using my debug map when I was trying to figure out what was going on :)

I just tested RFC 1918 IPs and `0.0.0.0`. Cymru returns the IP in the response, so the map should work! ``` Raw Value: NA | 127.0.0.1 | NA | |...