Merge pull request #18 from newcontext-oss/namecharacters-bugfix3

Reject -name values containing characters CredHub doesn't support

commands.go

@@ -243,6 +243,10 @@ func (v *GenerateAndStoreCommand) validateFlags() error {
 	if v.CommonName == "" && len(v.SANDNS) == 0 {
 		return errors.New("you must have a common name or san-dns")
 	}
+	re := regexp.MustCompile(`[^\w_\-\/]`)
+	if re.MatchString(v.Name) {
+		return errors.New("CredHub name can only contain alphanumeric characters plus forward slash, underscore and dash")
+	}
 	return nil
 }
 

cv.go

@@ -250,12 +250,12 @@ func (c CertCompareData) String() string {
 	if c.Left != nil {
 		out += fmt.Sprintf(" Left:%+v ", *c.Left)
 	} else {
-		out += fmt.Sprintf(" Left: nil ")
+		out += " Left: nil "
 	}
 	if c.Right != nil {
 		out += fmt.Sprintf(" Right:%+v ", *c.Right)
 	} else {
-		out += fmt.Sprintf(" Right: nil ")
+		out += " Right: nil "
 	}
 	return out
 }