Errors if log_prefix, certificate_arn not set about tf_aws_alb HOT 4 CLOSED

Hey @hasbro-ielo , thanks for the issue submission. Breaking them down:

re: log_prefix - I'll need to do some further testing to see if terraform is able to get around something like this. It might be that if log_bucket is given, log_prefix also needs to be provided, even if it's a blank variable. There are some real limitations around what can be done within an attribute (like the access_logs block). It might just be an opportunity to clarify the docs on what is actually required.

Re: certificate_arn - this is the issue: the module expects as the default behavior to use HTTPS but that requires a cert. A better out of the box experience (which is more in line with the docs and easier to do) would be defaulting to HTTP. That's a super easy fix I can submit in the next week.

from tf_aws_alb.

Thank you very much for explaining the bit about log_prefix needing to be blank.

from tf_aws_alb.

Sorry for letting this languish. I'm transitioning workplaces at the moment and will have the time to focus on this next week.

from tf_aws_alb.

I think there's a fix here that could be derived from the new locals feature. I'll migrate this issue over to the new repo and plan to look at it shortly.

from tf_aws_alb.