User-defined environment vars should be loaded after ones defined in this module

[dmattia]

Right now, there is no way to override environment variables this module sets.

This is especially problematic given that ENCRYPTION_KEY is always the random string: https://github.com/tryretool/terraform-retool-modules/blob/main/modules/aws_ecs/locals.tf#L42-L45

I would like a way to customize the ENCRYPTION_KEY, but this does not seem possible with this module right now

[samsternatretool]

@dmattia thanks for the feedback! Two questions:

1. Can you elaborate a bit more on why you need to customize the encryption key? I want to make sure I understand your use case.
2. Would you be willing to submit a PR implementing this change?

[lynnwellhausen]

@samsternatretool here is the ticket that documents our discovery of the issue. I think that will be helpful for giving you greater context.

[samsternatretool]

@lynnwellhausen thanks for providing that intercom link, that gives me a lot more background. I'm not personally familiar with this behavior but it does seem wrong to me, I'll ask around the team.

In the meantime if you or anyone on your end wants to send a PR with a proposed fix (even if it's just a partial fix that you'd like us to complete), I'd be happy to review.