fix(grpc): use Numbers to represent long values

Configure gRPC to use Numbers to represent long values instead of Strings. This ensures that we can do accurate handling of numerical values. Fix #727
LN-Zap · Aug 29, 2018 · 47b95a0 · 47b95a0
1 parent 2931eff
commit 47b95a0
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/app/lib/lnd/lightning.js b/app/lib/lnd/lightning.js
@@ -71,7 +71,7 @@ class Lightning {
       // See https://github.com/grpc/grpc-node/blob/master/packages/grpc-protobufjs/README.md
       const options = {
         keepCase: true,
-        longs: String,
+        longs: Number,
         enums: String,
         defaults: true,
         oneofs: true

diff --git a/app/lib/lnd/walletUnlocker.js b/app/lib/lnd/walletUnlocker.js
@@ -21,7 +21,7 @@ export const walletUnlocker = lndConfig => {
   // See https://github.com/grpc/grpc-node/blob/master/packages/grpc-protobufjs/README.md
   const options = {
     keepCase: true,
-    longs: String,
+    longs: Number,
     enums: String,
     defaults: true,
     oneofs: true