[WIP] Fix ban by software version, some other under-hood changes

- Use updated version of upstream go-xmpp + fixes on client-side
- Still waiting for https://github.com/xmppo/go-xmpp/pull/215
- Introduce golangci-lint v2 + fixes

internal/jabber/buny.go

@@ -9,7 +9,7 @@ import (
 	"strings"
 
 	"github.com/davecgh/go-spew/spew"
-	"github.com/eleksir/go-xmpp"
+	"github.com/xmppo/go-xmpp"
 )
 
 // BunyPresense производит проверку по бело-чёрным спискам. Если presence пришёл от злодея (из чёрного списка), то
@@ -419,21 +419,22 @@ func (j *Jabber) BunySoftwareVersion(v xmpp.IQ, ver IqResultSoftwareVersion) err
 
 			_ = json.Unmarshal([]byte(presenceJSONString), &p)
 
-			if p.Affiliation == "admin" || p.Affiliation == "owner" {
-				log.Infof("Skipping useragent blacklist check for %s because his affiliation is %s", v.From, p.Affiliation)
-
-				continue
-			}
-
-			// На всякий случай.
-			if p.Role == "moderator" {
-				log.Infof("Skipping useragent blacklist check for %s because his role is %s", v.From, p.Role)
-
-				continue
-			}
-
 			if v.From == p.From {
-				log.Errorf("Found jid of %s in presence db: %s", v.From, p.JID)
+
+				log.Infof("Found jid of %s in presence db: %s", v.From, p.JID)
+
+				if p.Affiliation == "admin" || p.Affiliation == "owner" {
+					log.Infof("Skipping useragent blacklist check for %s because his affiliation is %s", v.From, p.Affiliation)
+
+					continue
+				}
+
+				// На всякий случай.
+				if p.Role == "moderator" {
+					log.Infof("Skipping useragent blacklist check for %s because his role is %s", v.From, p.Role)
+
+					continue
+				}
 
 				// Lookup in global blacklist.
 				for _, cRoom := range j.RoomsConnected {

internal/jabber/commands.go

@@ -7,7 +7,7 @@ import (
 	"strings"
 
 	"github.com/davecgh/go-spew/spew"
-	"github.com/eleksir/go-xmpp"
+	"github.com/xmppo/go-xmpp"
 )
 
 // Cmd парсит команды из чятика.

internal/jabber/event_parser.go

@@ -5,14 +5,13 @@ import (
 	"encoding/json"
 	"encoding/xml"
 	"fmt"
-	"runtime"
 	"slices"
 	"sort"
 	"strings"
 	"time"
 
 	"github.com/davecgh/go-spew/spew"
-	"github.com/eleksir/go-xmpp"
+	"github.com/xmppo/go-xmpp"
 )
 
 // ParseEvent парсит ивенты, прилетающие из модуля xmpp (изменения presence, фразы участников, ошибки).
@@ -113,66 +112,6 @@ func (j *Jabber) ParseEvent(e any) { //nolint:maintidx,gocognit,gocyclo
 
 			if err := xml.Unmarshal(v.Query, &iqStruct); err == nil {
 				switch iqStruct.Xmlns { //nolint:wsl,whitespace
-				// Запрос номера версии приложения.
-				case "jabber:iq:version":
-					log.Infof("Got IQ get request for version from %s", v.From)
-
-					if id, err := j.Talk.IqVersionResponse(
-						v,
-						j.C.ExeName,
-						j.C.Version,
-						(strings.SplitN(runtime.GOOS, "/", 2))[0],
-					); err != nil {
-						err := fmt.Errorf(
-							"unable to send version info to jabber server: id=%s, err=%w",
-							id,
-							err,
-						)
-
-						j.GTomb.Kill(err)
-
-						return
-					}
-
-					parseError = false
-
-				// Нам прислали попингуй.
-				case "urn:xmpp:ping":
-					log.Infof("Got IQ get request for pong from %s", v.From)
-
-					if id, err := j.Talk.PingResponse(v); err != nil {
-						err := fmt.Errorf(
-							"unable to send pong to jabber server: id=%s, err=%w",
-							id,
-							err,
-						)
-
-						j.GTomb.Kill(err)
-
-						return
-					}
-
-					parseError = false
-
-				// У нас запросили время последней активности. В нашем случае это либо время запуска клиента, либо время
-				// последней фразы в чяти. Xep-0012.
-				case "jabber:iq:last":
-					log.Infof("Got IQ get last activity time request from %s", v.From)
-
-					if id, err := j.Talk.JabberIqLastResponse(v, j.LastActivity); err != nil {
-						err := fmt.Errorf(
-							"unable to send last activity time to jabber server: id=%s, err=%w",
-							id,
-							err,
-						)
-
-						j.GTomb.Kill(err)
-
-						return
-					}
-
-					parseError = false
-
 				// Запрос на список поддерживаемых фич - версия, локальное время, итд.
 				case xmpp.XMPPNS_DISCO_INFO:
 					log.Infof("Got IQ get disco#info request from %s", v.From)
@@ -312,31 +251,6 @@ func (j *Jabber) ParseEvent(e any) { //nolint:maintidx,gocognit,gocyclo
 				}
 			}
 
-			// Попробуем распарсить входящий запрос как urn:xmpp:ping он приходит при MUC Self-Ping (Schrödinger's Chat).
-			// Предполагается, что такие респонсы должны приходить только для пинга без серверной оптимизации.
-			// Хотя идейно мы не поддерживаем работу без серверной оптимизации, но на пинг ответим, нам несложно.
-			if parseError {
-				var iqStruct IqPing
-
-				if err := xml.Unmarshal(v.Query, &iqStruct); err == nil {
-					log.Debugf("Got IQ get request (actually, response) for MUC Self-Ping from %s", v.From)
-
-					if id, err := j.Talk.PingResponse(v); err != nil {
-						err := fmt.Errorf(
-							"unable to send pong to jabber server: id=%s, err=%w",
-							id,
-							err,
-						)
-
-						j.GTomb.Kill(err)
-
-						return
-					}
-				}
-
-				parseError = false
-			}
-
 			// Не знаю, как парсить... залоггируем это дело и пойдём дальше.
 			if parseError {
 				log.Infof("Does not look like parsable via iqStruct")

internal/jabber/query_version.go

@@ -4,8 +4,8 @@ import (
 	"buny-jabber-bot/internal/log"
 	"fmt"
 
-	"github.com/eleksir/go-xmpp"
 	"github.com/google/uuid"
+	"github.com/xmppo/go-xmpp"
 )
 
 // QuerySoftwareVersion запрошивает версию и название клиента.

internal/jabber/sig_handler.go

@@ -5,7 +5,7 @@ import (
 	"os"
 	"syscall"
 
-	"github.com/eleksir/go-xmpp"
+	"github.com/xmppo/go-xmpp"
 )
 
 // SigHandler срабатывает если происходит завершение программы, инициированное либо init-ом, либо пользователем.

internal/jabber/squash.go

@@ -4,7 +4,7 @@ import (
 	"fmt"
 	"time"
 
-	"github.com/eleksir/go-xmpp"
+	"github.com/xmppo/go-xmpp"
 )
 
 // Squash банит указанный jid в указанной комнате.

internal/jabber/types.go

@@ -4,7 +4,7 @@ import (
 	"encoding/xml"
 	"os"
 
-	"github.com/eleksir/go-xmpp"
+	"github.com/xmppo/go-xmpp"
 	"gopkg.in/tomb.v2"
 )
 

internal/jabber/util.go

@@ -10,7 +10,7 @@ import (
 	"strings"
 	"time"
 
-	"github.com/eleksir/go-xmpp"
+	"github.com/xmppo/go-xmpp"
 )
 
 // RandomPhrase выдаёт одну рандомную фразу из даденного списка фраз.
@@ -125,7 +125,7 @@ func (j *Jabber) RotateStatus(room string) error {
 func (j *Jabber) JoinMuc(room string) error {
 	log.Debugf("Sending disco#info from %s to %s", j.Talk.JID(), room)
 
-	if _, err := j.Talk.DiscoverInfo(j.Talk.JID(), room); err != nil {
+	if _, err := j.Talk.DiscoverInfo(j.Talk.JID()); err != nil {
 		return fmt.Errorf("unable to send disco#info to MUC %s: %w", room, err)
 	}
 
@@ -251,7 +251,7 @@ func (j *Jabber) EstablishConnection() error {
 
 	log.Debugf("Sending disco#info to %s", j.C.Jabber.Server)
 
-	_, err = j.Talk.DiscoverInfo(j.Talk.JID(), j.C.Jabber.Server)
+	_, err = j.Talk.DiscoverInfo(j.Talk.JID())
 
 	if err != nil {
 		return fmt.Errorf("unable to send disco#info to jabber server: %w", err)