[thrift] branch master updated: THRIFT-3926: Fix undeclared statusCode variable (#1531)

[jk...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

jking pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/thrift.git


The following commit(s) were added to refs/heads/master by this push:
     new 63b75d9  THRIFT-3926: Fix undeclared statusCode variable (#1531)
63b75d9 is described below

commit 63b75d9e3a4112b5b278933b487095f7bb431c1c
Author: Filip Spiridonov <fi...@gmail.com>
AuthorDate: Fri Apr 6 05:56:54 2018 -0700

    THRIFT-3926: Fix undeclared statusCode variable (#1531)
    
    Client: nodejs
---
 lib/nodejs/lib/thrift/http_connection.js | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/lib/nodejs/lib/thrift/http_connection.js b/lib/nodejs/lib/thrift/http_connection.js
index 4f5378f..3c2ab0f 100644
--- a/lib/nodejs/lib/thrift/http_connection.js
+++ b/lib/nodejs/lib/thrift/http_connection.js
@@ -170,7 +170,7 @@ var HttpConnection = exports.HttpConnection = function(options) {
     var dataLen = 0;
 
     if (response.statusCode !== 200) {
-      this.emit("error", new THTTPException(statusCode, response));
+      this.emit("error", new THTTPException(response));
     }
 
     response.on('error', function (e) {
@@ -251,11 +251,11 @@ exports.createHttpUDSConnection = function(path, options) {
 exports.createHttpClient = createClient
 
 
-function THTTPException(statusCode, response) {
+function THTTPException(response) {
   thrift.TApplicationException.call(this);
   Error.captureStackTrace(this, this.constructor);
   this.name = this.constructor.name;
-  this.statusCode = statusCode;
+  this.statusCode = response.statusCode;
   this.response = response;
   this.type = thrift.TApplicationExceptionType.PROTOCOL_ERROR;
   this.message = "Received a response with a bad HTTP status code: " + response.statusCode;

-- 
To stop receiving notification emails like this one, please contact
jking@apache.org.