Add additional logging (#18)

Should help us debug unnamed ikeSAs and all the connects and disconnects that
our metrics indicate.

internal/strongswan/strongswan.go

@@ -54,20 +54,25 @@ func collectSasStats(configs []map[string]vici.IKEConf, sas []map[string]vici.Ik
 		}
 	}
 
-	log.Infof("Sas: %d", len(sas))
 	for _, sa := range sas {
 		for ikeName, ikeSa := range sa {
 			conf, ok := expectedConnections[ikeName]
 			if !ok {
 				log.Errorf("Unexpected SA: %s: %#v", ikeName, ikeSa)
 				continue
 			}
-			log.Infof("  ikeName: %s: sa: %#v", ikeName, ikeSa)
+			log.With("conf", conf).
+				With("sa", ikeSa).
+				With("ikeName", ikeName).
+				Infof("Reporting on ike name '%s'", ikeName)
 			reporter.IKESAStatus(ikeName, conf, &ikeSa)
 			delete(expectedConnections, ikeName)
 		}
 	}
 	for ikeName, conf := range expectedConnections {
+		log.With("conf", conf).
+			With("ikeName", ikeName).
+			Infof("Reporting config without active SAs on '%s'", ikeName)
 		reporter.IKESAStatus(ikeName, conf, nil)
 	}
 }

internal/tcpchecker/logreporter.go

@@ -13,30 +13,30 @@ type logReporter struct {
 }
 
 func (r *logReporter) ReportPortCheck(report Report) {
+	l := r.Logger.With("report", report)
 	switch {
 	case report.Open && r.lastOpen:
 		// Port is still open - great
 	case !report.Open && (r.lastOpen || r.lastReport == time.Time{}):
 		// Port closed
 		r.lastReport = time.Now()
 		r.lastOpen = report.Open
-		r.Logger.
+		l.
 			With("status", "closed").
 			Infof("TCP connection to %s closed", report.Name)
 	case report.Open && !r.lastOpen:
 		// Port opened
 		r.lastReport = time.Now()
 		r.lastOpen = report.Open
-		r.Logger.
+		l.
 			With("status", "opened").
-			With("content", report.Content).
 			Infof("TCP connection to %s opened", report.Name)
 	case !report.Open && !r.lastOpen:
 		// Port still closed
 		if time.Since(r.lastReport) > 5*time.Minute {
 			r.lastReport = time.Now()
 			r.lastOpen = report.Open
-			r.Logger.
+			l.
 				With("status", "closed").
 				Infof("TCP connection to %s is still closed", report.Name)
 		}

internal/tcpchecker/tcpchecker.go

@@ -53,7 +53,7 @@ func Check(name string, address string, port int, reporter Reporter) {
 				Address: address,
 				Port:    port,
 				Open:    true,
-				Status:  "Open",
+				Status:  "Open (closed by us)",
 				Error:   nil,
 				Content: output.String(),
 			})
@@ -75,7 +75,7 @@ func Check(name string, address string, port int, reporter Reporter) {
 		Address: address,
 		Port:    port,
 		Open:    true,
-		Status:  "Open",
+		Status:  "Open (closed by peer)",
 		Error:   nil,
 		Content: output.String(),
 	})