fix server panic on ping

Author: Speshal71

server.go

@@ -58,9 +58,8 @@ func (s *IprotoServer) WithOptions(opts *IprotoServerOptions) *IprotoServer {
 		opts = &IprotoServerOptions{}
 	}
 	s.perf = opts.Perf
-	s.getPingStatus = opts.GetPingStatus
-	if s.getPingStatus == nil {
-		s.getPingStatus = func(*IprotoServer) uint { return 0 }
+	if opts.GetPingStatus != nil {
+		s.getPingStatus = opts.GetPingStatus
 	}
 	return s
 }
@@ -229,7 +228,9 @@ READER_LOOP:
 				code := packet.Cmd
 				if code == PingCommand {
 					pr := packetPool.GetWithID(packet.requestID)
-					pr.packet.Cmd = s.getPingStatus(s)
+					if s.getPingStatus != nil {
+						pr.packet.Cmd = s.getPingStatus(s)
+					}
 					pr.packet.SchemaID = packet.SchemaID
 
 					select {

server_test.go

@@ -0,0 +1,44 @@
+package tarantool
+
+import (
+	"context"
+	"net"
+	"testing"
+
+	"github.com/stretchr/testify/assert"
+	"github.com/stretchr/testify/require"
+)
+
+func TestServerPing(t *testing.T) {
+	handler := func(queryContext context.Context, query Query) *Result {
+		return &Result{}
+	}
+
+	s := NewIprotoServer("1", handler, nil)
+
+	listenAddr := make(chan string)
+	go func() {
+		ln, err := net.Listen("tcp", "127.0.0.1:0")
+		require.NoError(t, err)
+		defer ln.Close()
+
+		listenAddr <- ln.Addr().String()
+		close(listenAddr)
+
+		conn, err := ln.Accept()
+		require.NoError(t, err)
+
+		s.Accept(conn)
+	}()
+
+	addr := <-listenAddr
+	conn, err := Connect(addr, nil)
+	require.NoError(t, err)
+
+	res := conn.Exec(context.Background(), &Ping{})
+	assert.Equal(t, res.ErrorCode, OKCommand)
+	assert.NoError(t, res.Error)
+
+	conn.Close()
+	s.Shutdown()
+}