made it a single function

Author: lobsterjerusalem

config/config.go

@@ -168,14 +168,7 @@ func NewRemoteExploit(implemented ImplementedFeatures, extype ExploitType, suppo
 	product []string, cpe []string, cve string, protocol string, defaultPort int,
 ) *Config {
 	newConf := new(Config)
-
-	joinedProducts := strings.Join(product, "/")
-	if joinedProducts == vendor {
-		newConf.Product = vendor // prevents awkward duplicate word output such as "Validating Product Product"
-	} else {
-		newConf.Product = fmt.Sprintf("%s %s", vendor, joinedProducts)
-	}
-
+	newConf.Product = deDupeProductName(product, vendor)
 	newConf.InitFlagsStructs()
 	newConf.Impl = implemented
 	newConf.ExType = extype
@@ -191,19 +184,21 @@ func NewRemoteExploit(implemented ImplementedFeatures, extype ExploitType, suppo
 	return newConf
 }
 
-// Defines a new remote exploit and associates with CVE/Product/Protocol metadata. Usage example:.
-func NewLocalExploit(implemented ImplementedFeatures, extype ExploitType, supportedC2 []c2.Impl, vendor string,
-	product []string, cpe []string, cve string,
-) *Config {
-	newConf := new(Config)
-
+func deDupeProductName(product []string, vendor string) string {
 	joinedProducts := strings.Join(product, "/")
 	if joinedProducts == vendor {
-		newConf.Product = vendor // prevents awkward duplicate word output such as "Validating Product Product"
+		return vendor
 	} else {
-		newConf.Product = fmt.Sprintf("%s %s", vendor, joinedProducts)
+		return fmt.Sprintf("%s %s", vendor, joinedProducts)
 	}
+}
 
+// Defines a new remote exploit and associates with CVE/Product/Protocol metadata. Usage example:.
+func NewLocalExploit(implemented ImplementedFeatures, extype ExploitType, supportedC2 []c2.Impl, vendor string,
+	product []string, cpe []string, cve string,
+) *Config {
+	newConf := new(Config)
+	newConf.Product = deDupeProductName(product, vendor)
 	newConf.InitFlagsStructs()
 	newConf.Impl = implemented
 	newConf.ExType = extype