fix(v2): http biz error should unmarshal JSON

Author: crispgm

v2/api_notification_test.go

@@ -86,7 +86,7 @@ func TestWebhookSignedError(t *testing.T) {
 	mbText := NewMsgBuffer(MsgText)
 	mbText.Text("hello sign").WithSign("LIpnNexV7rwOyOebKoqSdb", time.Now().Unix())
 	resp, err := bot.PostNotification(t.Context(), mbText.Build())
-	assert.NoError(t, err)
+	assert.Error(t, err)
 	assert.Zero(t, resp.StatusCode)
 	assert.Equal(t, "sign match fail or timestamp is not within one hour from current time", resp.Msg)
 }

v2/http.go

@@ -73,6 +73,12 @@ func (bot Bot) PerformAPIRequest(
 		bot.httpErrorLog(ctx, prefix, "read body failed", err)
 		return err
 	}
+	// read response content
+	err = json.Unmarshal(buffer, &output)
+	if err != nil {
+		bot.httpErrorLog(ctx, prefix, "decode body failed", err)
+		return err
+	}
 	// read error code
 	var dummyOutput DummyResponse
 	err = json.Unmarshal(buffer, &dummyOutput)
@@ -81,11 +87,6 @@ func (bot Bot) PerformAPIRequest(
 		bot.logger.Log(ctx, LogLevelError, apiError.Error())
 		return apiError
 	}
-	// read response content
-	err = json.Unmarshal(buffer, &output)
-	if err != nil {
-		bot.httpErrorLog(ctx, prefix, "decode body failed", err)
-	}
 	return err
 }