From 3bd85a5798b1cc9719b0514ee53e6baad38f1622 Mon Sep 17 00:00:00 2001
From: Charles Gagnon <chgagnon@microsoft.com>
Date: Fri, 21 Apr 2023 13:27:11 -0700
Subject: [PATCH] Move stringifying of request body for azure REST calls
 (#22820)

* Move stringifying of request body for azure REST calls

* spelling

* Remove unused
---
 extensions/azurecore/src/account-provider/auths/httpClient.ts | 2 +-
 extensions/azurecore/src/azureResource/utils.ts               | 3 ++-
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/extensions/azurecore/src/account-provider/auths/httpClient.ts b/extensions/azurecore/src/account-provider/auths/httpClient.ts
index aa9e1b115e..d44a340c12 100644
--- a/extensions/azurecore/src/account-provider/auths/httpClient.ts
+++ b/extensions/azurecore/src/account-provider/auths/httpClient.ts
@@ -286,7 +286,7 @@ const networkRequestViaHttps = <T>(
 	const isPostRequest = httpMethod === HttpMethod.POST;
 	const isPutRequest = httpMethod === HttpMethod.PUT;
 	// Note: Text Encoder is necessary here because otherwise it was not able to handle Chinese characters in table names.
-	const body = (new TextEncoder()).encode(JSON.stringify(options?.body || ''));
+	const body = (new TextEncoder()).encode(options?.body || '');
 	const url = new URL(urlString);
 	const optionHeaders = options?.headers || {} as Record<string, string>;
 	let customOptions: https.RequestOptions = {
diff --git a/extensions/azurecore/src/azureResource/utils.ts b/extensions/azurecore/src/azureResource/utils.ts
index 6079216ad6..d0a89c3a2a 100644
--- a/extensions/azurecore/src/azureResource/utils.ts
+++ b/extensions/azurecore/src/azureResource/utils.ts
@@ -396,9 +396,10 @@ export async function makeHttpRequest(account: AzureAccount, subscription: azure
 		...requestHeaders
 	}
 
+	const body = JSON.stringify(requestBody || '');
 	let networkRequestOptions: NetworkRequestOptions = {
 		headers: reqHeaders,
-		body: requestBody
+		body
 	};
 
 	// Adding '/' if path does not begin with it.