Merge pull request #313 from crazy-max/fix-docker-api

crazy-max · web-flow · commit 8ee2a4721d35 · 2024-01-31T12:09:38.000+01:00
docker: enables automatic API version negotiation for the client
diff --git a/internal/docker/client.go b/internal/docker/client.go
@@ -34,7 +34,7 @@ type DockerClient struct {
 
 // NewEnvClient initializes a new Docker API client based on environment variables
 func NewEnvClient() (*DockerClient, error) {
-	dockerAPICli, err := client.NewClientWithOpts(client.FromEnv, client.WithVersion("1.12"))
+	dockerAPICli, err := client.NewClientWithOpts(client.FromEnv, client.WithAPIVersionNegotiation())
 	if err != nil {
 		return nil, errors.Wrap(err, "failed to initialize Docker API client")
 	}

Original file line number	Diff line number	Diff line change
`@@ -34,7 +34,7 @@ type DockerClient struct {`
`34`	`34`
`35`	`35`	`// NewEnvClient initializes a new Docker API client based on environment variables`
`36`	`36`	`func NewEnvClient() (*DockerClient, error) {`
`37`		`- dockerAPICli, err := client.NewClientWithOpts(client.FromEnv, client.WithVersion("1.12"))`
	`37`	`+ dockerAPICli, err := client.NewClientWithOpts(client.FromEnv, client.WithAPIVersionNegotiation())`
`38`	`38`	`if err != nil {`
`39`	`39`	`return nil, errors.Wrap(err, "failed to initialize Docker API client")`
`40`	`40`	`}`