fingerprint-container-node-sdk - npm Package Compare versions

Comparing version 0.1.2 to 0.1.3

client.js

@@ -14,7 +14,13 @@ var util = require('util');
 
-Client.prototype.rpcRequest = function (serviceName, body, callback, timeout) {
+Client.prototype.rpcRequest = function (serviceName, body, success, error, timeout) {
     var _me = this;
     var msg = this._genMsg(serviceName, {flag: FLAG.RPC});
+    var hasErrorFn = typeof error == "function";
     Service.prototype.rpcRequest.call(this, body, msg);
 
+    if (!hasErrorFn) {
+        timeout = error;
+        error = null;
+    }
+
     var timer = setTimeout(function () {
@@ -24,3 +30,4 @@ var askId = msg.askId;
         _me.rpcs.splice(rpcInfo.index, 1);
-        _me.emit('timeout', {
+        hasErrorFn && error({
+            error: "timeout",
             serviceName: serviceName,
@@ -34,3 +41,3 @@ body: body,
         askId: msg.askId,
-        callback: callback,
+        callback: success,
         timer: timer,
@@ -37,0 +44,0 @@ index: this.rpcs.length

package.json

 {
   "name": "fingerprint-container-node-sdk",
-  "version": "0.1.2",
+  "version": "0.1.3",
   "description": "fingerprint container's sdk",
@@ -5,0 +5,0 @@ "main": "fpc.js",

README.md

@@ -59,5 +59,7 @@ FPC NODE SDK
             console.log(body);
-        }, 2000);
+        }, function(){
+            console.log("timeout");
+        },2000);
     });
 ```
 

test/client.js

@@ -6,3 +6,3 @@ var assert = require('assert');
 
-var SERVICE_HOST = '172.16.50.46';
+var SERVICE_HOST = '172.16.90.42';
 var SERVICE_PORT = 9900;
@@ -116,9 +116,7 @@
                     name: serviceName
-                });
-
-                setTimeout(function () {
-                    if (!client.rpcs.length) {
-                        done();
-                    }
-                }, 4000);
+                }, function () {
+                    
+                }, function () {
+                    done();
+                }, 2000);
             });
@@ -125,0 +123,0 @@ });

test/server.js

@@ -5,3 +5,3 @@ var assert = require('assert');
 
-var SERVICE_HOST = '172.16.50.46';
+var SERVICE_HOST = '172.16.90.42';
 var SERVICE_PORT = 9900;
@@ -8,0 +8,0 @@

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

31266

increased by0.82%

Lines of code

741

increased by0.54%

Number of lines in readme file

65

increased by3.17%

No dependency changes