add Logging section to README.

remove global vars proxies, host

.gitignore

@@ -19,3 +19,7 @@
 
 # Go workspace file
 go.work
+
+# testing
+run.sh
+server.yaml

README.md

@@ -2,6 +2,10 @@
 
 A modification of [lilproxy][lilproxy_url] that forwards only Q3 rcon/query packets. Useful for separating the rcon port from the game server port.
 
+### Why
+
+Unfortunately the Q3Rcon engine ties the rcon port to the game servers public port used for client connections. This proxy will allow you to run rcon through a separate whitelisted port.
+
 ### Use
 
 Run one or multiple rcon proxies by setting an environment variable `Q3RCON_PROXY`
@@ -16,9 +20,11 @@ This would configure q3rcon-proxy to run 3 proxy servers listening on ports `200
 
 Then just run the binary which you can compile yourself, download from `Releases` or use the included Dockerfile.
 
-### Why
+### Logging
 
-Avoid sending plaintext rcon commands to the public game server port. In general I would advise anyone using rcon remotely to use a secured connection but perhaps you've passed rcon to a clan friend who doesn't know about secured connections. Now you can instruct them to use rcon only through a whitelisted port.
+Set the log level with environment variable `Q3RCON_LOGLEVEL`:
+
+`0 = Panic, 1 = Fatal, 2 = Error, 3 = Warning, 4 = Info, 5 = Debug, 6 = Trace`
 
 ### Special Thanks
 

cmd/q3rcon-proxy/main.go

@@ -3,7 +3,7 @@ package main
 import (
 	"fmt"
 	"os"
-	"strconv"
+	"slices"
 	"strings"
 
 	log "github.com/sirupsen/logrus"
@@ -11,7 +11,33 @@ import (
 	"github.com/onyx-and-iris/q3rcon-proxy/pkg/udpproxy"
 )
 
-func start(proxy string) {
+func main() {
+	logLevel, err := getEnvInt("Q3RCON_LOGLEVEL")
+	if err != nil {
+		log.Fatalf("unable to parse Q3RCON_LEVEL: %s", err.Error())
+	}
+	if slices.Contains(log.AllLevels, log.Level(logLevel)) {
+		log.SetLevel(log.Level(logLevel))
+	}
+
+	proxies := os.Getenv("Q3RCON_PROXY")
+	if proxies == "" {
+		log.Fatal("env Q3RCON_PROXY required")
+	}
+
+	host := os.Getenv("Q3RCON_HOST")
+	if host == "" {
+		host = "0.0.0.0"
+	}
+
+	for _, proxy := range strings.Split(proxies, ";") {
+		go start(host, proxy)
+	}
+
+	<-make(chan int)
+}
+
+func start(host, proxy string) {
 	port, target := func() (string, string) {
 		x := strings.Split(proxy, ":")
 		return x[0], x[1]
@@ -26,51 +52,3 @@ func start(proxy string) {
 
 	log.Fatal(c.ListenAndServe())
 }
-
-var (
-	proxies, host string
-)
-
-func getenvInt(key string) (int, error) {
-	s := os.Getenv(key)
-	if s == "" {
-		return 0, nil
-	}
-	v, err := strconv.Atoi(s)
-	if err != nil {
-		return 0, err
-	}
-	return v, nil
-}
-
-func init() {
-	proxies = os.Getenv("Q3RCON_PROXY")
-	if proxies == "" {
-		log.Fatal("env Q3RCON_PROXY required")
-	}
-
-	host = os.Getenv("Q3RCON_HOST")
-	if host == "" {
-		host = "0.0.0.0"
-	}
-
-	debug, err := getenvInt("Q3RCON_DEBUG")
-	if err != nil {
-		log.Fatal(err)
-	}
-
-	if debug == 1 {
-		log.SetLevel(log.DebugLevel)
-	} else {
-		log.SetLevel(log.InfoLevel)
-	}
-
-}
-
-func main() {
-	for _, proxy := range strings.Split(proxies, ";") {
-		go start(proxy)
-	}
-
-	<-make(chan int)
-}

cmd/q3rcon-proxy/util.go

@@ -0,0 +1,18 @@
+package main
+
+import (
+	"os"
+	"strconv"
+)
+
+func getEnvInt(key string) (int, error) {
+	s := os.Getenv(key)
+	if s == "" {
+		return 0, nil
+	}
+	v, err := strconv.Atoi(s)
+	if err != nil {
+		return 0, err
+	}
+	return v, nil
+}

pkg/udpproxy/session.go

@@ -2,6 +2,7 @@ package udpproxy
 
 import (
 	"errors"
+	"fmt"
 	"net"
 	"strings"
 	"time"
@@ -68,7 +69,7 @@ func (s *session) proxyFrom(buf []byte) error {
 		if s.isBadRconResponse(buf) {
 			log.Infof("Response: Bad rcon from %s", s.caddr.IP)
 		} else {
-			log.Debugf("Response: %s", string(buf[10:]))
+			log.Debugf("Response: %s", string(buf[len(s.rconResponseHeader):]))
 		}
 	}
 
@@ -77,7 +78,13 @@ func (s *session) proxyFrom(buf []byte) error {
 
 func (s *session) proxyTo(buf []byte) error {
 	if !s.isValidRequestPacket(buf) {
-		err := errors.New("not a rcon or query request packet")
+		var err error
+		if s.isChallengeRequestPacket(buf) {
+			parts := strings.SplitN(string(buf), " ", 3)
+			err = fmt.Errorf("invalid challenge from %s with GUID: %s", s.caddr.IP, parts[len(parts)-1])
+		} else {
+			err = errors.New("not a rcon or query request packet")
+		}
 		log.Error(err.Error())
 		return err
 	}

pkg/udpproxy/udpproxy.go

@@ -53,8 +53,8 @@ func (c *Client) ListenAndServe() error {
 			log.Error(err)
 		}
 
-		session, found := c.sessions[caddr.String()]
-		if !found {
+		session, ok := c.sessions[caddr.String()]
+		if !ok {
 			session, err = newSession(caddr, c.raddr, c.proxyConn)
 			if err != nil {
 				log.Error(err)

pkg/udpproxy/validator.go

@@ -3,57 +3,63 @@ package udpproxy
 import "bytes"
 
 type validator struct {
-	rconRequestHeader       []byte
-	getstatusRequestHeader  []byte
-	getinfoRequestHeader    []byte
-	rconResponseHeader      []byte
-	getstatusResponseHeader []byte
-	getinfoResponseHeader   []byte
-	badRconIdentifier       []byte
+	rconRequestHeader         []byte
+	getstatusRequestHeader    []byte
+	getinfoRequestHeader      []byte
+	getchallengeRequestHeader []byte
+	rconResponseHeader        []byte
+	getstatusResponseHeader   []byte
+	getinfoResponseHeader     []byte
+	badRconIdentifier         []byte
 }
 
 func newValidator() validator {
-	v := validator{}
-	v.rconRequestHeader = []byte("\xff\xff\xff\xffrcon")
-	v.getstatusRequestHeader = []byte("\xff\xff\xff\xffgetstatus")
-	v.getinfoRequestHeader = []byte("\xff\xff\xff\xffgetinfo")
-	v.rconResponseHeader = []byte("\xff\xff\xff\xffprint\n")
-	v.getstatusResponseHeader = []byte("\xff\xff\xff\xffstatusResponse\n")
-	v.getinfoResponseHeader = []byte("\xff\xff\xff\xffinfoResponse\n")
-	v.badRconIdentifier = []byte("Bad rcon")
-	return v
+	return validator{
+		rconRequestHeader:         []byte("\xff\xff\xff\xffrcon"),
+		getstatusRequestHeader:    []byte("\xff\xff\xff\xffgetstatus"),
+		getinfoRequestHeader:      []byte("\xff\xff\xff\xffgetinfo"),
+		getchallengeRequestHeader: []byte("\xff\xff\xff\xffgetchallenge"),
+		rconResponseHeader:        []byte("\xff\xff\xff\xffprint\n"),
+		getstatusResponseHeader:   []byte("\xff\xff\xff\xffstatusResponse\n"),
+		getinfoResponseHeader:     []byte("\xff\xff\xff\xffinfoResponse\n"),
+		badRconIdentifier:         []byte("Bad rcon"),
+	}
 }
 
-func (v *validator) compare(buf, c []byte) bool {
+func (v validator) compare(buf, c []byte) bool {
 	return bytes.Equal(buf[:len(c)], c)
 }
 
-func (v *validator) isRconRequestPacket(buf []byte) bool {
+func (v validator) isRconRequestPacket(buf []byte) bool {
 	return v.compare(buf, v.rconRequestHeader)
 }
 
-func (v *validator) isQueryRequestPacket(buf []byte) bool {
+func (v validator) isQueryRequestPacket(buf []byte) bool {
 	return v.compare(buf, v.getstatusRequestHeader) ||
 		v.compare(buf, v.getinfoRequestHeader)
 }
 
-func (v *validator) isValidRequestPacket(buf []byte) bool {
+func (v validator) isValidRequestPacket(buf []byte) bool {
 	return v.isRconRequestPacket(buf) || v.isQueryRequestPacket(buf)
 }
 
-func (v *validator) isRconResponsePacket(buf []byte) bool {
+func (v validator) isChallengeRequestPacket(buf []byte) bool {
+	return v.compare(buf, v.getchallengeRequestHeader)
+}
+
+func (v validator) isRconResponsePacket(buf []byte) bool {
 	return v.compare(buf, v.rconResponseHeader)
 }
 
-func (v *validator) isQueryResponsePacket(buf []byte) bool {
+func (v validator) isQueryResponsePacket(buf []byte) bool {
 	return v.compare(buf, v.getstatusResponseHeader) ||
 		v.compare(buf, v.getinfoResponseHeader)
 }
 
-func (v *validator) isValidResponsePacket(buf []byte) bool {
+func (v validator) isValidResponsePacket(buf []byte) bool {
 	return v.isRconResponsePacket(buf) || v.isQueryResponsePacket(buf)
 }
 
-func (v *validator) isBadRconResponse(buf []byte) bool {
+func (v validator) isBadRconResponse(buf []byte) bool {
 	return v.compare(buf[len(v.rconResponseHeader):], v.badRconIdentifier)
 }