Properly handle the nil response on read in raft_snapshot_agent_config resource (#1360)

- Ensure that a nil response on read for raft_snapshot_agent_config
removes the resource from the state.

Author: benashz

CHANGELOG.md

@@ -2,6 +2,7 @@
 BUGS:
 * `resource/identity_group`: Report an error upon duplicate resource creation failure. Document group name caveats. ([#1352](https://github.com/hashicorp/terraform-provider-vault/pull/1352))
 * `resource/pki_secret_backend_root_sign_intermediate`: Fix panic when reading `ca_chain` from Vault ([#1357](https://github.com/hashicorp/terraform-provider-vault/issues/1357)
+* `resource/raft_snapshot_agent_config`: Properly handle nil response on read ([#1360](https://github.com/hashicorp/terraform-provider-vault/pull/1360))
 
 ## 3.3.0 (February 17, 2022)
 FEATURES:

vault/resource_raft_snapshot_agent_config.go

@@ -11,8 +11,10 @@ import (
 	"github.com/hashicorp/vault/api"
 )
 
-var snapshotAutoPath = "sys/storage/raft/snapshot-auto/config/%s"
-var allowedStorageTypes = []string{"local", "azure-blob", "aws-s3", "google-gcs"}
+var (
+	snapshotAutoPath    = "sys/storage/raft/snapshot-auto/config/%s"
+	allowedStorageTypes = []string{"local", "azure-blob", "aws-s3", "google-gcs"}
+)
 
 func raftSnapshotAgentConfigResource() *schema.Resource {
 	fields := map[string]*schema.Schema{
@@ -196,7 +198,6 @@ func buildConfigFromResourceData(d *schema.ResourceData) (map[string]interface{}
 		} else {
 			return nil, errors.New("specified local storage without setting local_max_space")
 		}
-
 	}
 
 	if storageType == "aws-s3" {
@@ -306,11 +307,12 @@ func readSnapshotAgentConfigResource(d *schema.ResourceData, meta interface{}) e
 	log.Printf("[DEBUG] Reading %q", configPath)
 
 	resp, err := client.Logical().Read(configPath)
-	if err != nil && util.Is404(err) {
+	if resp == nil || (err != nil && util.Is404(err)) {
 		log.Printf("[WARN] %q not found, removing from state", name)
 		d.SetId("")
 		return nil
-	} else if err != nil {
+	}
+	if err != nil {
 		return fmt.Errorf("error reading %q: %s", configPath, err)
 	}