lightning 7.1.2 → 7.1.3

package/CHANGELOG.md

@@ -1,6 +1,6 @@
 # Versions
 
-## 7.1.2
+## 7.1.3
 
 - `signChainAddressMessage`: Add method to sign a message given a chain address
 - `verifyChainAddressMessage`: Add method to verify a chain address message

package/lnd_methods/onchain/broadcast_chain_transaction.js

@@ -7,6 +7,7 @@ const {isLnd} = require('./../../lnd_requests');
 const bufFromHex = hex => Buffer.from(hex, 'hex');
 const {fromHex} = Transaction;
 const method = 'publishTransaction';
+const minRelayFeeError = /^unmatched backend error: -26: mempool min fee not met, (\d*) < (\d*)$/;
 const type = 'wallet';
 
 /** Publish a raw blockchain transaction to Blockchain network peers
@@ -51,6 +52,19 @@ module.exports = ({description, lnd, transaction}, cbk) => {
           tx_hex: bufFromHex(transaction),
         },
         (err, res) => {
+          if (!!err && minRelayFeeError.test(err.details)) {
+            const [, got, expected] = err.details.match(minRelayFeeError);
+
+            return cbk([
+              503,
+              'ChainBackendMinimumRelayFeeNotMet',
+              {
+                fee: Number(got),
+                minimum: Number(expected),
+              }
+            ]);
+          }
+
           if (!!err) {
             return cbk([503, 'UnexpectedErrBroadcastingRawTx', {err}]);
           }

package/package.json

@@ -7,10 +7,10 @@
     "url": "https://github.com/alexbosworth/lightning/issues"
   },
   "dependencies": {
-    "@grpc/grpc-js": "1.8.11",
+    "@grpc/grpc-js": "1.8.12",
     "@grpc/proto-loader": "0.7.5",
     "@types/express": "4.17.17",
-    "@types/node": "18.14.4",
+    "@types/node": "18.14.6",
     "@types/request": "2.48.8",
     "@types/ws": "8.5.4",
     "async": "3.2.4",
@@ -32,7 +32,7 @@
   "devDependencies": {
     "@alexbosworth/node-fetch": "2.6.2",
     "@alexbosworth/tap": "15.0.12",
-    "tsd": "0.25.0",
+    "tsd": "0.26.1",
     "typescript": "4.9.5",
     "ws": "8.12.1"
   },
@@ -59,5 +59,5 @@
     "directory": "test/typescript"
   },
   "types": "index.d.ts",
-  "version": "7.1.2"
+  "version": "7.1.3"
 }

package/test/lnd_methods/onchain/test_broadcast_chain_transaction.js

@@ -46,6 +46,27 @@ const tests = [
       {res: {publish_error: 'err'}},
     ],
   },
+  {
+    args: {
+      lnd: {
+        wallet: {
+          publishTransaction: ({}, cbk) => cbk({
+            details: 'unmatched backend error: -26: mempool min fee not met, 123 < 1234',
+          }),
+        },
+      },
+      transaction: new Transaction().toHex(),
+    },
+    description: 'Minimum relay fee not met',
+    error: [
+      503,
+      'ChainBackendMinimumRelayFeeNotMet',
+      {
+        fee: 123,
+        minimum: 1234,
+      },
+    ],
+  },
   {
     args: {
       lnd: {wallet: {publishTransaction: ({}, cbk) => cbk(null, {})}},

package/test/lnd_methods/signer/test_begin_group_signing_session.js

@@ -5,7 +5,7 @@ const {beginGroupSigningSession} = require('./../../../');
 const makeLnd = (err, res) => {
   return {
     signer: {
-      muSig2CreateSession: ({}, cbk) => cbk(err, res)
+      muSig2CreateSession: ({}, cbk) => cbk(err, res),
     },
     wallet: {
       deriveKey: ({}, cbk) => cbk(null, {
@@ -84,6 +84,153 @@ const tests = [
     description: 'A valid response is expected',
     error: [503, 'ExpectedCombinedPublicKeyInMuSig2SessionResponse'],
   },
+  {
+    args: makeArgs({
+      lnd: {
+        signer: {
+          muSig2CreateSession: (args, cbk) => {
+            if (args.version === 'MUSIG2_VERSION_V100RC2') {
+              return cbk({
+                details: 'error parsing signer public key 0: bad pubkey byte string size (want 32, have 33)',
+              });
+            }
+
+            return cbk({
+              details: 'unknown method MuSig2CreateSession for service signrpc.Signer',
+            });
+          },
+        },
+        wallet: {
+          deriveKey: ({}, cbk) => cbk(null, {
+            key_loc: {key_index: 0},
+            raw_key_bytes: Buffer.alloc(1),
+          }),
+        },
+      },
+    }),
+    description: 'Just in case unsupported message is supported',
+    error: [501, 'MuSig2BeginSigningSessionNotSupported'],
+  },
+  {
+    args: makeArgs({
+      lnd: {
+        signer: {
+          muSig2CreateSession: (args, cbk) => {
+            if (args.version === 'MUSIG2_VERSION_V100RC2') {
+              return cbk({
+                details: 'error parsing signer public key 0: bad pubkey byte string size (want 32, have 33)',
+              });
+            }
+
+            return cbk('err');
+          },
+        },
+        wallet: {
+          deriveKey: ({}, cbk) => cbk(null, {
+            key_loc: {key_index: 0},
+            raw_key_bytes: Buffer.alloc(1),
+          }),
+        },
+      },
+    }),
+    description: 'Errors are passed back',
+    error: [503, 'UnexpectedErrorCreatingMuSig2Session', {err: 'err'}],
+  },
+  {
+    args: makeArgs({
+      lnd: {
+        signer: {
+          muSig2CreateSession: (args, cbk) => {
+            if (args.version === 'MUSIG2_VERSION_V100RC2') {
+              return cbk({
+                details: 'error parsing signer public key 0: bad pubkey byte string size (want 32, have 33)',
+              });
+            }
+
+            return cbk(null, {});
+          },
+        },
+        wallet: {
+          deriveKey: ({}, cbk) => cbk(null, {
+            key_loc: {key_index: 0},
+            raw_key_bytes: Buffer.alloc(1),
+          }),
+        },
+      },
+    }),
+    description: 'A valid response is expected',
+    error: [503, 'ExpectedCombinedPublicKeyInMuSig2SessionResponse'],
+  },
+  {
+    args: makeArgs({
+      lnd: {
+        signer: {
+          muSig2CreateSession: (args, cbk) => {
+            if (args.version === 'MUSIG2_VERSION_V100RC2') {
+              return cbk({
+                details: 'error parsing signer public key 0: bad pubkey byte string size (want 32, have 33)',
+              });
+            }
+
+            return cbk(null, {
+              combined_key: Buffer.alloc(32, 1),
+              local_public_nonces: Buffer.alloc(66, 1),
+              session_id: Buffer.alloc(32, 1),
+              taproot_internal_key: Buffer.alloc(0),
+            });
+          },
+        },
+        wallet: {
+          deriveKey: ({}, cbk) => cbk(null, {
+            key_loc: {key_index: 0},
+            raw_key_bytes: Buffer.alloc(1),
+          }),
+        },
+      },
+    }),
+    description: 'Legacy signer is supported',
+    expected: {
+      external_key: '0101010101010101010101010101010101010101010101010101010101010101',
+      id: '0101010101010101010101010101010101010101010101010101010101010101',
+      internal_key: undefined,
+      nonce: '010101010101010101010101010101010101010101010101010101010101010101010101010101010101010101010101010101010101010101010101010101010101',
+    },
+  },
+  {
+    args: makeArgs({
+      lnd: {
+        signer: {
+          muSig2CreateSession: (args, cbk) => {
+            if (args.version === 'MUSIG2_VERSION_V100RC2') {
+              return cbk({
+                details: 'error parsing all signer public keys: error parsing signer public key 0 for v1.0.0rc2 (compressed format): malformed public key: invalid length: 32',
+              });
+            }
+
+            return cbk(null, {
+              combined_key: Buffer.alloc(32, 1),
+              local_public_nonces: Buffer.alloc(66, 1),
+              session_id: Buffer.alloc(32, 1),
+              taproot_internal_key: Buffer.alloc(0),
+            });
+          },
+        },
+        wallet: {
+          deriveKey: ({}, cbk) => cbk(null, {
+            key_loc: {key_index: 0},
+            raw_key_bytes: Buffer.alloc(1),
+          }),
+        },
+      },
+    }),
+    description: 'Legacy signer is supported when invalid keys specified',
+    expected: {
+      external_key: '0101010101010101010101010101010101010101010101010101010101010101',
+      id: '0101010101010101010101010101010101010101010101010101010101010101',
+      internal_key: undefined,
+      nonce: '010101010101010101010101010101010101010101010101010101010101010101010101010101010101010101010101010101010101010101010101010101010101',
+    },
+  },
   {
     args: makeArgs({}),
     description: 'A group signing session is started',