search

Security-Onion-Solutions / securityonion-cloud

GNU Affero General Public License v3.0
32 stars 16 forks source link

fixed it grabbing an object instead of the id(int value) #17

Closed jinwoov closed 3 years ago

jinwoov commented 3 years ago

Minor fixed in the /terraform/aws/ubuntu.tf and /terraform/aws/windows.tf.

In line 24, if logic had yield object instead of image id, so it was causing an error, so added .id to grab numerical image value to assign ami variable.

weslambert commented 3 years ago

Thanks for your contribution! Could you please ensure that the commit is signed, then re-submit a PR?

jinwoov commented 3 years ago

Good point will do!