From 943e8e648a01aae928bf2611cefa20ec47cdced8 Mon Sep 17 00:00:00 2001 From: Matthew Hooker Date: Mon, 7 Aug 2017 11:09:40 -0700 Subject: [PATCH] properly handle decode error --- post-processor/vagrant-cloud/step_release_version.go | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/post-processor/vagrant-cloud/step_release_version.go b/post-processor/vagrant-cloud/step_release_version.go index 73ba9b6f8..cc4db611f 100644 --- a/post-processor/vagrant-cloud/step_release_version.go +++ b/post-processor/vagrant-cloud/step_release_version.go @@ -31,7 +31,10 @@ func (s *stepReleaseVersion) Run(state multistep.StateBag) multistep.StepAction if err != nil || (resp.StatusCode != 200) { cloudErrors := &VagrantCloudErrors{} - _ = decodeBody(resp, cloudErrors) + if err := decodeBody(resp, cloudErrors); err != nil { + state.Put("error", fmt.Errorf("Error parsing provider response: %s", err)) + return multistep.ActionHalt + } if strings.Contains(cloudErrors.FormatErrors(), "already been released") { ui.Message("Not releasing version, already released") return multistep.ActionContinue