Merge pull request #10 from base2Services/feature/ssm-param-ami-lookup-fix

Guslington · web-flow · commit 99eb25c07a5e · 2021-10-07T15:22:21.000+11:00
fix issue where the the path was being validiated instead of the ami id
diff --git a/bastion/ami.go b/bastion/ami.go
@@ -27,7 +27,7 @@ func GetAndValidateAmi(sess *session.Session, input string) (string, error) {
 		if err != nil {
 			return "", err
 		}
-		if !ValidAmi(input) {
+		if !ValidAmi(ami) {
 			return "", errors.New("parameter value is not a valid ami")
 		}
 		return ami, nil

Original file line number	Diff line number	Diff line change
`@@ -27,7 +27,7 @@ func GetAndValidateAmi(sess *session.Session, input string) (string, error) {`
`27`	`27`	`if err != nil {`
`28`	`28`	`return "", err`
`29`	`29`	`}`
`30`		`- if !ValidAmi(input) {`
	`30`	`+ if !ValidAmi(ami) {`
`31`	`31`	`return "", errors.New("parameter value is not a valid ami")`
`32`	`32`	`}`
`33`	`33`	`return ami, nil`