azure-arm-rest (#322)

- moved HttpClient to method call
- added dispose method to HttpClient

Author: DmitriiBobreshev

common-npm-packages/azure-arm-rest/package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "azure-pipelines-tasks-azure-arm-rest",
-  "version": "3.241.1",
+  "version": "3.241.2",
   "description": "Common Lib for Azure ARM REST apis",
   "repository": {
     "type": "git",

common-npm-packages/azure-arm-rest/package.json

@@ -20,7 +20,7 @@ var requestOptions: httpInterfaces.IRequestOptions = proxyUrl ? {
 let ignoreSslErrors: string = tl.getVariable("VSTS_ARM_REST_IGNORE_SSL_ERRORS");
 requestOptions.ignoreSslError = ignoreSslErrors && ignoreSslErrors.toLowerCase() == "true";
 
-var httpCallbackClient = new httpClient.HttpClient(tl.getVariable("AZURE_HTTP_USER_AGENT"), null, requestOptions);
+var azureHttpUserAgent = tl.getVariable("AZURE_HTTP_USER_AGENT");
 
 export class WebRequest {
     public method: string;
@@ -112,8 +112,13 @@ export function sleepFor(sleepDurationInSeconds): Promise<any> {
 
 async function sendRequestInternal(request: WebRequest): Promise<WebResponse> {
     tl.debug(util.format("[%s]%s", request.method, request.uri));
+    var httpCallbackClient = new httpClient.HttpClient(azureHttpUserAgent, null, requestOptions);
+    
     var response: httpClient.HttpClientResponse = await httpCallbackClient.request(request.method, request.uri, request.body, request.headers);
-    return await toWebResponse(response);
+    const weResponse = await toWebResponse(response);
+    
+    httpCallbackClient.dispose();
+    return weResponse;
 }
 
 async function toWebResponse(response: httpClient.HttpClientResponse): Promise<WebResponse> {