Use default send after for empty string (#7683) (#5460)

Signed-off-by: Modular Magician <[email protected]>
hashicorp · Apr 11, 2023 · 1813abd · 1813abd
1 parent ab40b17
commit 1813abd
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/.changelog/7683.txt b/.changelog/7683.txt
@@ -0,0 +1,3 @@
+```release-note:bug
+all: fixed an error resulting from leaving `provider.batching.send_after` unspecified and `provider.batching` specified
+```
diff --git a/google-beta/config.go b/google-beta/config.go
@@ -1256,7 +1256,7 @@ func ExpandProviderBatchingConfig(v interface{}) (*batchingConfig, error) {
 	}
 
 	cfgV := ls[0].(map[string]interface{})
-	if sendAfterV, ok := cfgV["send_after"]; ok {
+	if sendAfterV, ok := cfgV["send_after"]; ok && sendAfterV != "" {
 		SendAfter, err := time.ParseDuration(sendAfterV.(string))
 		if err != nil {
 			return nil, fmt.Errorf("unable to parse duration from 'send_after' value %q", sendAfterV)