Update vendors + allow default_ttl to be set to 0 #41
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
As described in #20 and hashicorp/terraform#13648 and hashicorp/terraform#12910, not being able to set the
default_ttl
to0
is quite an unexpected behavior.To be able to do that we have to update the
github.com/ajg/form
dependency (I usedmake updatedeps
so all have been updated, tested and had to modifyfastly_test.go
to fix an incompatibility), which is the 1st commit of this PR.The 2nd removes the omitempty which removes the parameter when set to 0 and change the form encoder to keep zero values. Added a test for it.
Let me know if that's acceptable.
Thanks!