Use iterative logic to determine missing flag

context.go

@@ -313,18 +313,21 @@ func checkRequiredFlags(flags []Flag, context *Context) requiredFlagsErr {
 	var missingFlags []string
 	for _, f := range flags {
 		if rf, ok := f.(RequiredFlag); ok && rf.IsRequired() {
-			key := strings.Split(f.GetName(), ",")
-			if len(key) > 1 {
-				// has short name
-				if !context.IsSet(strings.TrimSpace(key[0])) && !context.IsSet(strings.TrimSpace(key[1])) {
-					missingFlags = append(missingFlags, key[0])
+			var flagPresent bool
+			var flagName string
+			for _, key := range strings.Split(f.GetName(), ",") {
+				if len(key) > 1 {
+					flagName = key
 				}
-			} else {
-				// does not have short name
-				if !context.IsSet(strings.TrimSpace(key[0])) {
-					missingFlags = append(missingFlags, key[0])
+
+				if context.IsSet(strings.TrimSpace(key)) {
+					flagPresent = true
 				}
 			}
+
+			if !flagPresent {
+				missingFlags = append(missingFlags, flagName)
+			}
 		}
 	}
 

context_test.go

@@ -524,6 +524,13 @@ func TestCheckRequiredFlags(t *testing.T) {
 			},
 			parseInput: []string{"-N", "asd", "-N", "qwe"},
 		},
+		{
+			testCase: "required_flag_with_short_name",
+			flags: []Flag{
+				StringSliceFlag{Name: "names, N, n", Required: true},
+			},
+			parseInput: []string{"-n", "asd", "-n", "qwe"},
+		},
 	}
 	for _, test := range tdata {
 		t.Run(test.testCase, func(t *testing.T) {