Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

user_data in aws_launch_configuration can't be gzip/base64 #497

Closed
hashibot opened this issue Jun 13, 2017 · 3 comments
Closed

user_data in aws_launch_configuration can't be gzip/base64 #497

hashibot opened this issue Jun 13, 2017 · 3 comments
Labels
documentation Introduces or discusses updates to documentation.

Comments

@hashibot
Copy link

This issue was originally opened by @dodwmd as hashicorp/terraform#11488. It was migrated here as part of the provider split. The original body of the issue is below.


Hi there,

Not really a bug. Most likely could be dealt with a documentation update to state that user_data in a aws_launch_configuration shouldn't be encoded when using template_cloudinit_config and doing multi-part configs.

@hashibot hashibot added the documentation Introduces or discusses updates to documentation. label Jun 13, 2017
@Ninir
Copy link
Contributor

Ninir commented Aug 10, 2017

Hey @dodwmd,

Martin worked on providing such stuff here: #850, so I assume this can be closed.

@apparentlymart Do you confirm? if so, I'll let you close this one :)

@Ninir Ninir added the waiting-response Maintainers are waiting on response from community or contributor. label Aug 10, 2017
@apparentlymart
Copy link
Member

Thanks for pointing this out, @Ninir.

Indeed the change in #850 adds a new attribute user_data_base64 that takes data that's already base64-encoded, including the output of template_cloudinit_config when gzip = true and base64 = true.

In future we will deprecate and eventually remove the broken combination of gzip = true and base64 = false, because Terraform strings are treated as UTF-8 and so it's not safe to pass around general binary data, as can be see in my comment on #754.

So with all of that said, I'm going to close this. The new attribute is not yet included in a stable release, but it should be included in the next one. (Note that the AWS provider is now released separately from Terraform core, so it will not be necessary to upgrade Terraform itself to get this new feature.)

@ghost
Copy link

ghost commented Apr 11, 2020

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. Thanks!

@ghost ghost locked and limited conversation to collaborators Apr 11, 2020
@breathingdust breathingdust removed the waiting-response Maintainers are waiting on response from community or contributor. label Sep 17, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
documentation Introduces or discusses updates to documentation.
Projects
None yet
Development

No branches or pull requests

4 participants