From 556c09c2f4c818240bea62b2feae076d62daf243 Mon Sep 17 00:00:00 2001
From: Hector <dev@hsmith.org>
Date: Sun, 29 Aug 2021 11:26:40 +0100
Subject: [PATCH] fix handling of ping response

Update the code handling sending the ping command to the fail2ban server
to correctly handle the response. The response is of type `Tupple`, not
`[]string`.
---
 src/socket/fail2banSocket.go | 18 +++++++++++-------
 1 file changed, 11 insertions(+), 7 deletions(-)

diff --git a/src/socket/fail2banSocket.go b/src/socket/fail2banSocket.go
index 3df6d3c..907e5b9 100644
--- a/src/socket/fail2banSocket.go
+++ b/src/socket/fail2banSocket.go
@@ -1,9 +1,11 @@
 package socket
 
 import (
-	"github.com/kisielk/og-rek"
 	"log"
 	"net"
+
+	"github.com/kisielk/og-rek"
+	"github.com/nlpodyssey/gopickle/types"
 )
 
 type Fail2BanSocket struct {
@@ -18,9 +20,7 @@ func MustConnectToSocket(path string) *Fail2BanSocket {
 	}
 	return &Fail2BanSocket{
 		socket: c,
-		encoder: ogórek.NewEncoderWithConfig(c, &ogórek.EncoderConfig{
-			Protocol: 5,
-		}),
+		encoder: ogórek.NewEncoder(c),
 	}
 }
 
@@ -30,9 +30,13 @@ func (s *Fail2BanSocket) Ping() bool {
 		log.Printf("server ping failed: %v", err)
 		return false
 	}
-	if v, ok := response.([]string); ok && v[1] == "pong" {
-		return true
+
+	if t, ok := response.(*types.Tuple); ok {
+		if (*t)[1] == "pong" {
+			return true
+		}
+		log.Printf("unexpected response data: %s", t)
 	}
-	log.Printf("unexpected response from server: %v", response)
+	log.Printf("unexpected response format - cannot parse: %v", response)
 	return false
 }