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 LN-Zap#727
mrfelton · Aug 29, 2018 · c12ca87 · c12ca87
1 parent 2931eff
commit c12ca87
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