Merge 34a34e7 into backport/b-ping-logmon-process-on-reattach/quietly…

…-rational-owl

.changelog/24798.txt

@@ -0,0 +1,3 @@
+```release-note:bug
+drivers: validate logmon plugin during reattach
+```

client/logmon/plugin.go

@@ -59,6 +59,16 @@ func LaunchLogMon(logger hclog.Logger, reattachConfig *plugin.ReattachConfig) (L
 		return nil, nil, err
 	}
 
+	// Note: Similar to reattaching to executors, Go-plugin uses localhost
+	// ports on Windows. On reattach, it may attach to another process
+	// listening on that port. We should validate it is actually a plugin.
+	if conf.Reattach != nil {
+		if err := rpcClient.Ping(); err != nil {
+			logger.Warn("failed to ping plugin process during reattach", "error", err)
+			return nil, nil, err
+		}
+	}
+
 	l := raw.(LogMon)
 	return l, client, nil
 }