-
Notifications
You must be signed in to change notification settings - Fork 324
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
resource/gitlab_repository_file: Support auto base64 encoding for content #972
resource/gitlab_repository_file: Support auto base64 encoding for content #972
Conversation
This pull request has merge conflicts. Please rebase your branch onto |
2213453
to
5c2e9b7
Compare
Conflicts are resolved. Thank you! 😀 |
This pull request has merge conflicts. Please rebase your branch onto |
5c2e9b7
to
dc7ba1a
Compare
Conflicts are resolved. Thank you! 😀 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks good to me @timofurrer - great work!
dc7ba1a
to
5d665e5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approving after @PatrickRice-KSC
This functionality has been released in v3.13.0 of the Terraform GitLab Provider. Please see the Terraform documentation on provider versioning or reach out if you need any assistance upgrading. For further feature requests or bug reports with this functionality, please create a new GitHub issue. Thank you! |
This change sets allows to specify not-yet base64 encoded
content
and the resource will automatically base64 encode it for the API.This will make the plan match more readable and will solve the limitations described in #956
Initially I thought it's best to use a
DiffSuppressFunc
and always store thecontent
as base64. However, I think it's best to store it in the same format (text vs. plan) as given in the config. This ensures that access the resource content in the config produces deterministic results.Closes: #956