From c9ac49c7581878cdfccdfcf84c5a11b9751f776b Mon Sep 17 00:00:00 2001 From: Cory Rivera Date: Fri, 15 Mar 2019 16:52:08 -0700 Subject: [PATCH] Check error in webhdfs.sendRequest before trying to check response code. (#4561) --- extensions/mssql/src/objectExplorerNodeProvider/webhdfs.ts | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/extensions/mssql/src/objectExplorerNodeProvider/webhdfs.ts b/extensions/mssql/src/objectExplorerNodeProvider/webhdfs.ts index 2843ad28c0..ec5a453b7c 100644 --- a/extensions/mssql/src/objectExplorerNodeProvider/webhdfs.ts +++ b/extensions/mssql/src/objectExplorerNodeProvider/webhdfs.ts @@ -213,11 +213,12 @@ export class WebHDFS { request(requestParams, (error, response, body) => { if (!callback) { return; } - if (this.isSuccess(response)) { - callback(undefined, response); - } else if (error || this.isError(response)) { + + if (error || this.isError(response)) { let hdfsError = this.parseError(response, body, error); callback(hdfsError, response); + } else if (this.isSuccess(response)) { + callback(undefined, response); } else { let hdfsError = new HdfsError( localize('webhdfs.unexpectedRedirect', 'Unexpected Redirect'),