fix: handle empty logs= value

internal/sshserver/connectionparams.go

@@ -10,7 +10,7 @@ import (
 var (
 	serviceRegex   = regexp.MustCompile(`^service=(.+)`)
 	containerRegex = regexp.MustCompile(`^container=(.+)`)
-	logsRegex      = regexp.MustCompile(`^logs=(.+)`)
+	logsRegex      = regexp.MustCompile(`^logs=(.+)?`)
 	tailLinesRegex = regexp.MustCompile(`^tailLines=(\d+)$`)
 )
 
@@ -117,6 +117,8 @@ func parseLogsArg(service, logs string, cmd []string) (bool, int64, error) {
 	for _, arg := range strings.Split(logs, ",") {
 		matches := tailLinesRegex.FindStringSubmatch(arg)
 		switch {
+		case arg == "":
+		// ignore empty arg
 		case arg == "follow":
 			follow = true
 		case len(matches) == 2:

internal/sshserver/connectionparams_test.go

@@ -74,6 +74,15 @@ func TestParseConnectionParams(t *testing.T) {
 				args:      []string{"drush do something"},
 			},
 		},
+		"service, container and empty logs params": {
+			input: []string{"service=nginx", "container=php", "logs=", "drush do something"},
+			expect: parsedParams{
+				service:   "nginx",
+				container: "php",
+				logs:      "",
+				args:      []string{"drush do something"},
+			},
+		},
 		"service, container and logs params (wrong order)": {
 			input: []string{"service=nginx", "logs=follow", "container=php", "drush do something"},
 			expect: parsedParams{
@@ -225,6 +234,13 @@ func TestValidateConnectionParams(t *testing.T) {
 				err: sshserver.ErrInvalidLogsValue,
 			},
 		},
+		"empty logs args - rely on default": {
+			input: parsedParams{
+				service: "nginx-php",
+				logs:    "",
+			},
+			expect: result{},
+		},
 	}
 	for name, tc := range testCases {
 		t.Run(name, func(tt *testing.T) {