PMM-13543 Fix socket path failure. (#969)

* PMM-13543 Fix socket path failure.

* PMM-13543 Address golangci review.

Author: BupycHuk

exporter/server.go

@@ -159,8 +159,9 @@ func OverallTargetsHandler(exporters []*Exporter, logger *logrus.Logger) http.Ha
 				registry.MustRegister(gc)
 			}
 
-			hostlabels := prometheus.Labels{
-				"instance": e.opts.NodeName,
+			hostlabels := prometheus.Labels{}
+			if e.opts.NodeName != "" {
+				hostlabels["instance"] = e.opts.NodeName
 			}
 
 			gw := NewGathererWrapper(registry, hostlabels)

main.go

@@ -140,14 +140,17 @@ func main() {
 }
 
 func buildExporter(opts GlobalFlags, uri string, log *logrus.Logger) *exporter.Exporter {
-	uri = buildURI(uri, opts.User, opts.Password, log)
+	uri = buildURI(uri, opts.User, opts.Password)
 	log.Debugf("Connection URI: %s", uri)
 
 	uriParsed, _ := url.Parse(uri)
 	var nodeName string
-	if uriParsed.Port() != "" {
+	switch {
+	case uriParsed == nil:
+		nodeName = ""
+	case uriParsed.Port() != "":
 		nodeName = net.JoinHostPort(uriParsed.Hostname(), uriParsed.Port())
-	} else {
+	default:
 		nodeName = uriParsed.Host
 	}
 
@@ -295,7 +298,7 @@ func buildURIManually(uri string, user string, password string) string {
 	return uri
 }
 
-func buildURI(uri string, user string, password string, log *logrus.Logger) string {
+func buildURI(uri string, user string, password string) string {
 	defaultPrefix := "mongodb://" // default prefix
 
 	if !strings.HasPrefix(uri, defaultPrefix) && !strings.HasPrefix(uri, "mongodb+srv://") {

main_test.go

@@ -272,7 +272,7 @@ func TestBuildURI(t *testing.T) {
 	}
 	for _, tc := range tests {
 		t.Run(tc.situation, func(t *testing.T) {
-			newURI := buildURI(tc.origin, tc.newUser, tc.newPassword, logrus.New())
+			newURI := buildURI(tc.origin, tc.newUser, tc.newPassword)
 			assert.Equal(t, tc.expect, newURI)
 		})
 	}