Sanitize optional param assume_role if provided

kciter / aws-ecr-action

This Action allows you to create Docker images and push into a ECR repository.

MIT License

146 stars 116 forks source link

Closed smitp closed 4 years ago

smitp commented 4 years ago

sanitizing empty assume_role results in error. Sanitizing it after presence check prevents it

bash-3.2$ sanitize "" "assume_role"
Unable to find the assume_role. Did you set with.assume_role?

mingchuno commented 4 years ago

@kciter

kciter commented 4 years ago

Thank you