add ports flag to k8scan

Signed-off-by: Jess Frazelle <acidburn@microsoft.com>

k8scan/Dockerfile

@@ -5,20 +5,23 @@ ENV PATH /go/bin:/usr/local/go/bin:$PATH
 ENV GOPATH /go
 
 RUN	apk add --no-cache \
-	ca-certificates
-
-COPY main.go /go/src/k8scan/
-
-RUN set -x \
-	&& apk add --no-cache --virtual .build-deps \
+	ca-certificates \
 	git \
 	gcc \
 	libc-dev \
 	libgcc \
-		make \
-	&& go get honnef.co/go/tools/cmd/staticcheck \
-	&& go get github.com/golang/lint/golint \
-	&& cd /go/src/k8scan/ \
+	make
+
+# Get go deps for tests etc.
+RUN go get honnef.co/go/tools/cmd/staticcheck \
+		github.com/golang/lint/golint \
+		github.com/google/go-cmp/cmp
+
+WORKDIR /go/src/k8scan
+
+COPY *.go /go/src/k8scan/
+
+RUN set -x \
 	&& go get -d . \
 	&& gofmt -s -l . \
 	&& go vet ./... \
@@ -26,8 +29,6 @@ RUN set -x \
 	&& staticcheck ./... \
 	&& go test ./... \
 	&& CGO_ENABLED=0 go build -a -tags netgo -ldflags '-w -extldflags "-static"' -o /usr/bin/k8scan *.go \
-	&& apk del .build-deps \
-	&& rm -rf /go \
 	&& echo "Build complete."
 
 FROM scratch

k8scan/main.go

@@ -36,7 +36,7 @@ var (
 
 	cidr string
 
-	defaultPorts = intSlice{80, 443, 9001, 8001}
+	defaultPorts = intSlice{80, 443, 8001, 9001}
 	ports        intSlice
 
 	mailgunDomain  string
@@ -89,7 +89,7 @@ func (i *intSlice) Set(value string) error {
 				*i = append(*i, port)
 			}
 
-			return nil
+			continue
 		}
 
 		// It is not a range just parse the port
@@ -272,7 +272,7 @@ type ARINResponse struct {
 	Net NetJSON `json:"net,omitempty"`
 }
 
-// NETJSON holds the net data from the ARIN response.
+// NetJSON holds the net data from the ARIN response.
 type NetJSON struct {
 	Organization OrganizationJSON `json:"orgRef,omitempty"`
 }

k8scan/main_test.go

@@ -1,8 +1,9 @@
 package main
 
 import (
-	"reflect"
 	"testing"
+
+	"github.com/google/go-cmp/cmp"
 )
 
 func TestARINResponse(t *testing.T) {
@@ -54,7 +55,7 @@ func TestParsePortRange(t *testing.T) {
 		if err := i.Set(testFunc.given); err != nil {
 			t.Fatal(err)
 		}
-		if reflect.DeepEqual(testFunc.expected, i) {
+		if !cmp.Equal(testFunc.expected, i) {
 			t.Fatalf("expected: %#v\ngot: %#v", testFunc.expected, i)
 		}
 	}