fixed govet issues

.golangci.yml

@@ -63,7 +63,6 @@ linters:
     - paralleltest
     - maintidx
     - ifshort
-    - gosimple
     - unparam
     - nakedret
     - misspell
@@ -93,7 +92,6 @@ linters:
     - unconvert
     - varcheck
     - errchkjson
-    - govet
     - nilnil
     - thelper
     - ireturn

cmd/frpc/sub/root.go

@@ -121,7 +121,9 @@ func Execute() {
 }
 
 func handleSignal(svr *client.Service) {
-	ch := make(chan os.Signal)
+	// We must use a buffered channel or risk missing the signal
+	// if we're not ready to receive when the signal is sent.
+	ch := make(chan os.Signal, 1)
 	signal.Notify(ch, syscall.SIGINT, syscall.SIGTERM)
 	<-ch
 	svr.GracefulClose(500 * time.Millisecond)

test/e2e/plugin/server.go

@@ -149,7 +149,7 @@ var _ = Describe("[Feature: Server-Plugins]", func() {
 			[tcp]
 			type = tcp
 			local_port = {{ .%s }}
-			remote_port = 0
+			remote_port = %d
 			`, framework.TCPEchoServerPort, remotePort)
 
 			f.RunProcesses([]string{serverConf}, []string{clientConf})