From 9fb5d0849b1075255d2ad1e2ee5a2b300b4484e2 Mon Sep 17 00:00:00 2001
From: Kevin Burke <kev@inburke.com>
Date: Sat, 17 Jun 2017 15:20:39 -0700
Subject: [PATCH] maintner: suppress spurious error message

Currently maintnerd prints

    2017/06/17 15:00:16 Unknown error type <nil>: <nil>

if it successfully downloads a file. It shouldn't log if there was no
error.

Change-Id: I01c4f2bde616d709d5c081219f9c123d6ea6a678
Reviewed-on: https://go-review.googlesource.com/46007
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
---
 maintner/netsource.go | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/maintner/netsource.go b/maintner/netsource.go
index 5c4d28f4..356743f0 100644
--- a/maintner/netsource.go
+++ b/maintner/netsource.go
@@ -74,6 +74,9 @@ func (ns *netMutSource) GetMutations(ctx context.Context) <-chan MutationStreamE
 // isNoInternetError reports whether the provided error is because there's no
 // network connectivity.
 func isNoInternetError(err error) bool {
+	if err == nil {
+		return false
+	}
 	switch err := err.(type) {
 	case *url.Error:
 		return isNoInternetError(err.Err)