Logical bug

when CLIENT_LIST exist its only do numberConnectedClient++ and not execute metricsCollect
kumina · Feb 20, 2018 · 5307c3c · 5307c3c
1 parent 8221d6d
commit 5307c3c
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/openvpn_exporter.go b/openvpn_exporter.go
@@ -198,9 +198,10 @@ func CollectServerStatusFromReader(statusPath string, file io.Reader, ch chan<-
 				statusPath)
 		} else if fields[0] == "TITLE" && len(fields) == 2 {
 			// OpenVPN version number.
-		} else if fields[0] == "CLIENT_LIST"{
-			numberConnectedClient ++
 		} else if header, ok := openvpnServerHeaders[fields[0]]; ok {
+			if fields[0] == "CLIENT_LIST"{
+				numberConnectedClient ++
+			}
 			// Entry that depends on a preceding HEADERS directive.
 			columnNames, ok := headersFound[fields[0]]
 			if !ok {