Very minor clean-ups (import re-ordering by automated tooling) (#28)

cmd/mc-router/metrics.go

@@ -4,6 +4,9 @@ import (
 	"context"
 	"errors"
 	"fmt"
+	"strings"
+	"time"
+
 	kitlogrus "github.com/go-kit/kit/log/logrus"
 	discardMetrics "github.com/go-kit/kit/metrics/discard"
 	expvarMetrics "github.com/go-kit/kit/metrics/expvar"
@@ -11,8 +14,6 @@ import (
 	influx "github.com/influxdata/influxdb1-client/v2"
 	"github.com/itzg/mc-router/server"
 	"github.com/sirupsen/logrus"
-	"strings"
-	"time"
 )
 
 type MetricsBuilder interface {

mcproto/read.go

@@ -4,14 +4,15 @@ import (
 	"bufio"
 	"bytes"
 	"encoding/binary"
-	"github.com/pkg/errors"
-	"github.com/sirupsen/logrus"
-	"golang.org/x/text/encoding/unicode"
-	"golang.org/x/text/transform"
 	"io"
 	"net"
 	"strings"
 	"time"
+
+	"github.com/pkg/errors"
+	"github.com/sirupsen/logrus"
+	"golang.org/x/text/encoding/unicode"
+	"golang.org/x/text/transform"
 )
 
 func ReadPacket(reader io.Reader, addr net.Addr, state State) (*Packet, error) {

mcproto/read_test.go

@@ -2,9 +2,10 @@ package mcproto
 
 import (
 	"bytes"
+	"testing"
+
 	"github.com/stretchr/testify/assert"
 	"github.com/stretchr/testify/require"
-	"testing"
 )
 
 func TestReadVarInt(t *testing.T) {

server/api_server.go

@@ -2,9 +2,10 @@ package server
 
 import (
 	"expvar"
+	"net/http"
+
 	"github.com/gorilla/mux"
 	"github.com/sirupsen/logrus"
-	"net/http"
 )
 
 var apiRoutes = mux.NewRouter()

server/connector.go

@@ -3,15 +3,16 @@ package server
 import (
 	"bytes"
 	"context"
+	"io"
+	"net"
+	"strconv"
+	"time"
+
 	"github.com/go-kit/kit/metrics"
 	"github.com/itzg/mc-router/mcproto"
 	"github.com/juju/ratelimit"
 	"github.com/pires/go-proxyproto"
 	"github.com/sirupsen/logrus"
-	"io"
-	"net"
-	"strconv"
-	"time"
 )
 
 const (
@@ -243,7 +244,6 @@ func (c *connectorImpl) findAndConnectBackend(ctx context.Context, frontendConn
 	}
 
 	c.pumpConnections(ctx, frontendConn, backendConn)
-	return
 }
 
 func (c *connectorImpl) pumpConnections(ctx context.Context, frontendConn, backendConn net.Conn) {

server/k8s.go

@@ -1,17 +1,18 @@
 package server
 
 import (
+	"net"
+	"strconv"
+	"strings"
+
 	"github.com/pkg/errors"
 	"github.com/sirupsen/logrus"
-	"k8s.io/api/core/v1"
+	v1 "k8s.io/api/core/v1"
 	"k8s.io/apimachinery/pkg/fields"
 	"k8s.io/client-go/kubernetes"
 	"k8s.io/client-go/rest"
 	"k8s.io/client-go/tools/cache"
 	"k8s.io/client-go/tools/clientcmd"
-	"net"
-	"strconv"
-	"strings"
 )
 
 const (

server/k8s_test.go

@@ -2,10 +2,11 @@ package server
 
 import (
 	"encoding/json"
+	"testing"
+
 	"github.com/stretchr/testify/assert"
 	"github.com/stretchr/testify/require"
 	v1 "k8s.io/api/core/v1"
-	"testing"
 )
 
 func TestK8sWatcherImpl_handleAddThenUpdate(t *testing.T) {

server/routes.go

@@ -2,11 +2,12 @@ package server
 
 import (
 	"encoding/json"
-	"github.com/gorilla/mux"
-	"github.com/sirupsen/logrus"
 	"net/http"
 	"strings"
 	"sync"
+
+	"github.com/gorilla/mux"
+	"github.com/sirupsen/logrus"
 )
 
 func init() {
@@ -134,17 +135,13 @@ func (r *routesImpl) FindBackendForServerAddress(serverAddress string) (string,
 
 	address := strings.ToLower(addressParts[0])
 
-	if r.mappings == nil {
-		return r.defaultRoute, address
-	} else {
-
+	if r.mappings != nil {
 		if route, exists := r.mappings[address]; exists {
 			return route, address
-		} else {
+		}
+	}
 	return r.defaultRoute, address
 }
-	}
-}
 
 func (r *routesImpl) GetMappings() map[string]string {
 	r.RLock()

server/routes_test.go

@@ -1,8 +1,9 @@
 package server
 
 import (
-	"github.com/stretchr/testify/assert"
 	"testing"
+
+	"github.com/stretchr/testify/assert"
 )
 
 func Test_routesImpl_FindBackendForServerAddress(t *testing.T) {