stricter pool validations

2 files changed, 23 insertions, 0 deletions

diff --git a/service/http/config.go b/service/http/config.go
index fb4574b1..76a0f539 100644
--- a/service/http/config.go
+++ b/service/http/config.go
@@ -34,6 +34,10 @@ func (cfg *Config) Valid() error {
 		return errors.New("mailformed workers config")
 	}
 
+	if cfg.Workers.Pool == nil {
+		return errors.New("mailformed workers config (pool config is missing)")
+	}
+
 	if !strings.Contains(cfg.Address, ":") {
 		return errors.New("mailformed server address")
 	}
diff --git a/service/http/config_test.go b/service/http/config_test.go
index 44c58f35..d47934ea 100644
--- a/service/http/config_test.go
+++ b/service/http/config_test.go
@@ -64,6 +64,25 @@ func Test_Config_NoWorkers(t *testing.T) {
 	assert.Error(t, cfg.Valid())
 }
 
+
+func Test_Config_NoPool(t *testing.T) {
+	cfg := &Config{
+		Enable:     true,
+		Address:    ":8080",
+		MaxRequest: 1024,
+		Uploads: &UploadsConfig{
+			Dir:    os.TempDir(),
+			Forbid: []string{".go"},
+		},
+		Workers: &roadrunner.ServerConfig{
+			Command: "php php-src/tests/client.php echo pipes",
+			Relay:   "pipes",
+		},
+	}
+
+	assert.Error(t, cfg.Valid())
+}
+
 func Test_Config_InvalidAddress(t *testing.T) {
 	cfg := &Config{
 		Enable:     true,