0.9.6: Added Validate() for DockerRun

This release adds a `Validate()` method for the DockerRun backend.

CHANGELOG.md

@@ -1,5 +1,9 @@
 # Changelog
 
+## 0.9.6: Added Validate() for DockerRun
+
+This release adds a `Validate()` method for the DockerRun backend.
+
 ## 0.9.5: Metrics integration
 
 This release integrates the [metrics library](https://github.com/containerssh/metrics) and adds two parameters to `New` and `NewDockerRun` methods:

config_dockerrun.go

@@ -3,12 +3,24 @@ package docker
 import (
 	"bytes"
 	"encoding/json"
+	"fmt"
 	"time"
 
 	"gopkg.in/yaml.v3"
 )
 
-//goland:noinspection GoNameStartsWithPackageName
+// Validate validates the docker run config
+//goland:noinspection GoDeprecation
+func (config DockerRunConfig) Validate() error {
+	if config.Host == "" {
+		return fmt.Errorf("empty Docker host provided")
+	}
+	if err := config.Config.Validate(); err != nil {
+		return fmt.Errorf("invalid dockerrun config (%w)", err)
+	}
+	return nil
+}
+
 //Deprecated: Switch to the more generic "docker" backend.
 //goland:noinspection GoNameStartsWithPackageName,GoDeprecation
 type DockerRunContainerConfig struct {
@@ -71,6 +83,12 @@ func (d *DockerRunContainerConfig) UnmarshalYAML(unmarshal func(interface{}) err
 	return nil
 }
 
+// Validate validates the container config
+//goland:noinspection GoDeprecation
+func (d *DockerRunContainerConfig) Validate() error {
+	return d.LaunchConfig.Validate()
+}
+
 type tmpDockerRunContainerConfig struct {
 	Subsystems     map[string]string `json:"subsystems" yaml:"subsystems" comment:"Subsystem names and binaries map." default:"{\"sftp\":\"/usr/lib/openssh/sftp-server\"}"`
 	DisableCommand bool              `json:"disableCommand" yaml:"disableCommand" comment:"Disable command execution passed from SSH"`

config_execution.go

@@ -82,7 +82,9 @@ func (c ExecutionConfig) Validate() error {
 	if err := c.ImagePullPolicy.Validate(); err != nil {
 		return err
 	}
-
+	if err := c.Launch.Validate(); err != nil {
+		return err
+	}
 	if err := c.Mode.Validate(); err != nil {
 		return err
 	}

config_launch.go

@@ -3,6 +3,7 @@ package docker
 import (
 	"bytes"
 	"encoding/json"
+	"fmt"
 
 	"github.com/docker/docker/api/types/container"
 	"github.com/docker/docker/api/types/network"
@@ -79,3 +80,14 @@ func (l *LaunchConfig) UnmarshalYAML(unmarshal func(interface{}) error) error {
 	l.ContainerName = tmp.ContainerName
 	return nil
 }
+
+// Validate validates the launch configuration.
+func (l *LaunchConfig) Validate() error {
+	if l.ContainerConfig == nil {
+		return fmt.Errorf("no container config provided")
+	}
+	if l.ContainerConfig.Image == "" {
+		return fmt.Errorf("no image name provided")
+	}
+	return nil
+}

factory_dockerrun.go

@@ -9,6 +9,7 @@ import (
 )
 
 // NewDockerRun creates a new NetworkConnectionHandler based on the deprecated "dockerrun" config structure.
+// Deprecated: use New instead
 //goland:noinspection GoDeprecation
 func NewDockerRun(
 	client net.TCPAddr,