Merge pull request #49 from form3tech-oss/jk-bump-deps

Bump go to 1.22 & all deps

build/package/pact-proxy/Dockerfile

@@ -1,4 +1,4 @@
-FROM golang:1.18-alpine AS build-env
+FROM golang:1.22-alpine AS build-env
 
 ARG APPNAME
 ENV GO111MODULE=auto

go.mod

@@ -1,39 +1,39 @@
 module github.com/form3tech-oss/pact-proxy
 
-go 1.18
+go 1.22
 
 require (
 	github.com/PaesslerAG/jsonpath v0.1.1
-	github.com/avast/retry-go/v4 v4.3.1
-	github.com/labstack/echo/v4 v4.9.1
-	github.com/pact-foundation/pact-go v1.7.0
+	github.com/avast/retry-go/v4 v4.5.1
+	github.com/labstack/echo/v4 v4.11.4
+	github.com/pact-foundation/pact-go v1.8.0
 	github.com/pkg/errors v0.9.1
-	github.com/sethvargo/go-envconfig v0.8.2
-	github.com/sirupsen/logrus v1.9.0
-	github.com/stretchr/testify v1.8.1
+	github.com/sethvargo/go-envconfig v1.0.1
+	github.com/sirupsen/logrus v1.9.3
+	github.com/stretchr/testify v1.9.0
 	github.com/tidwall/sjson v1.2.5
 )
 
 require (
-	github.com/PaesslerAG/gval v1.2.1 // indirect
+	github.com/PaesslerAG/gval v1.2.2 // indirect
 	github.com/davecgh/go-spew v1.1.1 // indirect
 	github.com/golang-jwt/jwt v3.2.2+incompatible // indirect
 	github.com/hashicorp/go-version v1.6.0 // indirect
 	github.com/hashicorp/logutils v1.0.0 // indirect
-	github.com/labstack/gommon v0.4.0 // indirect
+	github.com/labstack/gommon v0.4.2 // indirect
 	github.com/mattn/go-colorable v0.1.13 // indirect
-	github.com/mattn/go-isatty v0.0.16 // indirect
+	github.com/mattn/go-isatty v0.0.20 // indirect
 	github.com/pmezard/go-difflib v1.0.0 // indirect
 	github.com/shopspring/decimal v1.3.1 // indirect
-	github.com/tidwall/gjson v1.14.4 // indirect
+	github.com/tidwall/gjson v1.17.1 // indirect
 	github.com/tidwall/match v1.1.1 // indirect
 	github.com/tidwall/pretty v1.2.1 // indirect
 	github.com/valyala/bytebufferpool v1.0.0 // indirect
 	github.com/valyala/fasttemplate v1.2.2 // indirect
-	golang.org/x/crypto v0.3.0 // indirect
-	golang.org/x/net v0.2.0 // indirect
-	golang.org/x/sys v0.2.0 // indirect
-	golang.org/x/text v0.4.0 // indirect
-	golang.org/x/time v0.0.0-20201208040808-7e3f01d25324 // indirect
+	golang.org/x/crypto v0.22.0 // indirect
+	golang.org/x/net v0.24.0 // indirect
+	golang.org/x/sys v0.19.0 // indirect
+	golang.org/x/text v0.14.0 // indirect
+	golang.org/x/time v0.5.0 // indirect
 	gopkg.in/yaml.v3 v3.0.1 // indirect
 )

internal/app/configuration/proxyconfig.go

@@ -4,16 +4,20 @@ import (
 	"context"
 	"net/url"
 
-	"github.com/form3tech-oss/pact-proxy/internal/app/pactproxy"
 	"github.com/pkg/errors"
 	"github.com/sethvargo/go-envconfig"
+
+	"github.com/form3tech-oss/pact-proxy/internal/app/pactproxy"
 )
 
 func NewFromEnv() (pactproxy.Config, error) {
 	ctx := context.Background()
 
 	var config pactproxy.Config
-	err := envconfig.Process(ctx, &config)
+	err := envconfig.ProcessWith(ctx, &envconfig.Config{
+		Target:        &config,
+		DefaultNoInit: true,
+	})
 	if err != nil {
 		return config, errors.Wrap(err, "process env config")
 	}

internal/app/configuration/proxyconfig_test.go

@@ -21,9 +21,10 @@ import (
 	"testing"
 	"time"
 
-	"github.com/form3tech-oss/pact-proxy/internal/app/pactproxy"
 	"github.com/pact-foundation/pact-go/utils"
 	"github.com/stretchr/testify/require"
+
+	"github.com/form3tech-oss/pact-proxy/internal/app/pactproxy"
 )
 
 const (
@@ -183,6 +184,16 @@ func TestConfigureProxy_MTLS(t *testing.T) {
 	}
 }
 
+// When no server address env var is specified, config.ServerAddress should be the zero value.
+// This is used during ConfigureProxy logic, but is no longer the default behaviour of envconfig.Process due to url.URL
+// containing a pointer.
+func TestProxyConfig_NewFromEnv_EmptyServerAddress(t *testing.T) {
+	os.Clearenv()
+	config, err := NewFromEnv()
+	require.NoError(t, err)
+	require.Equal(t, url.URL{}, config.ServerAddress)
+}
+
 func TestProxyConfig_ForwardUnrecognisedRequests(t *testing.T) {
 	os.Clearenv()
 	config, err := NewFromEnv()