Merge pull request #123 from CiscoCloud/fix/portopen-typo

Fix typo where port number was appended to host
CiscoCloud · Jan 4, 2016 · ad8f8b9 · ad8f8b9
2 parents 91ea88e + 86ed9ab
commit ad8f8b9
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/netstatus/netstatus.go b/netstatus/netstatus.go
@@ -80,7 +80,7 @@ func PortOpen(protocol string, port uint16) bool {
 	if err != nil {
 		log.Fatal(err)
 	}
-	return CanConnect("localhost"+fmt.Sprint(port), protocol, dur)
+	return CanConnect(fmt.Sprintf("localhost:%v", port), protocol, dur)
 }
 
 // ValidIP returns a boolean answering the question "is this a valid IPV4/6

diff --git a/netstatus/netstatus_test.go b/netstatus/netstatus_test.go
@@ -54,7 +54,7 @@ func TestOpenPorts(t *testing.T) {
 				t.Errorf(msg+": %d", port)
 			} else {
 				dur, _ := time.ParseDuration("10s")
-				if CanConnect("localhost:"+fmt.Sprint(port), protocol, dur) {
+				if CanConnect(fmt.Sprintf("localhost:%v", port), protocol, dur) {
 					couldConnect++
 				}
 			}