edge-currency-plugins 1.1.4 → 1.2.0

package/lib/common/utxobased/engine/makeUtxoEngine.js

@@ -275,6 +275,8 @@ var _utils = require('./utils');
       options
     ) {
       const { spendTargets } = edgeSpendInfo
+      const { outputSort = 'bip69' } = _nullishCoalesce(edgeSpendInfo.otherParams, () => ( {}))
+
       if (_optionalChain([options, 'optionalAccess', _5 => _5.CPFP]) == null && spendTargets.length < 1) {
         throw new Error('Need to provide Spend Targets')
       }
@@ -397,7 +399,8 @@ var _utils = require('./utils');
         setRBF,
         freshChangeAddress,
         subtractFee,
-        log
+        log,
+        outputSort
       })
       if (tx.changeUsed) {
         ourReceiveAddresses.push(freshChangeAddress)

package/lib/common/utxobased/engine/makeUtxoEngineState.js

@@ -1,4 +1,5 @@
 "use strict";Object.defineProperty(exports, "__esModule", {value: true}); function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; } function _nullishCoalesce(lhs, rhsFn) { if (lhs != null) { return lhs; } else { return rhsFn(); } } async function _asyncNullishCoalesce(lhs, rhsFn) { if (lhs != null) { return lhs; } else { return await rhsFn(); } } function _optionalChain(ops) { let lastAccessLHS = undefined; let value = ops[0]; let i = 1; while (i < ops.length) { const op = ops[i]; const fn = ops[i + 1]; i += 2; if ((op === 'optionalAccess' || op === 'optionalCall') && value == null) { return undefined; } if (op === 'access' || op === 'optionalAccess') { lastAccessLHS = value; value = fn(value); } else if (op === 'call' || op === 'optionalCall') { value = fn((...args) => value.call(lastAccessLHS, ...args)); lastAccessLHS = undefined; } } return value; }var _biggystring = require('biggystring');
+var _cleaners = require('cleaners');
 
 
 
@@ -1109,6 +1110,42 @@ const processTransactionResponse = (
     txResponse,
     pluginInfo: { coinInfo }
   } = args
+  const inputs = txResponse.vin.map(vin => {
+    const scriptPubkey =
+      // Note: Blockbook has empirically not sent a hex value as the
+      // scriptPubkey for vins. If we discover this to be changed for some
+      // cases, we may want to use the `hex` field as an optimization.
+      _cleaners.asMaybe.call(void 0, 
+        raw => _utils3.validScriptPubkeyFromAddress.call(void 0, raw),
+        'unknown'
+      )({
+        address: vin.addresses[0],
+        coin: coinInfo.name
+      })
+    return {
+      txId: vin.txid,
+      outputIndex: vin.vout, // case for tx `fefac8c22ba1178df5d7c90b78cc1c203d1a9f5f5506f7b8f6f469fa821c2674` no `vout` for input
+      n: vin.n,
+      scriptPubkey,
+      amount: vin.value
+    }
+  })
+  const outputs = txResponse.vout.map(vout => {
+    const scriptPubkey =
+      _nullishCoalesce(vout.hex, () => (
+      _cleaners.asMaybe.call(void 0, 
+        raw => _utils3.validScriptPubkeyFromAddress.call(void 0, raw),
+        'unknown'
+      )({
+        address: vout.addresses[0],
+        coin: coinInfo.name
+      })))
+    return {
+      n: vout.n,
+      scriptPubkey,
+      amount: vout.value
+    }
+  })
   return {
     txid: txResponse.txid,
     hex: txResponse.hex,
@@ -1116,26 +1153,8 @@ const processTransactionResponse = (
     blockHeight: txResponse.blockHeight > 0 ? txResponse.blockHeight : 0,
     date: txResponse.blockTime,
     fees: txResponse.fees,
-    inputs: txResponse.vin.map(input => ({
-      txId: input.txid,
-      outputIndex: input.vout, // case for tx `fefac8c22ba1178df5d7c90b78cc1c203d1a9f5f5506f7b8f6f469fa821c2674` no `vout` for input
-      n: input.n,
-      scriptPubkey: _utils3.validScriptPubkeyFromAddress.call(void 0, {
-        address: input.addresses[0],
-        coin: coinInfo.name
-      }),
-      amount: input.value
-    })),
-    outputs: txResponse.vout.map(output => ({
-      n: output.n,
-      scriptPubkey:
-        _nullishCoalesce(output.hex, () => (
-        _utils3.validScriptPubkeyFromAddress.call(void 0, {
-          address: output.addresses[0],
-          coin: coinInfo.name
-        }))),
-      amount: output.value
-    })),
+    inputs,
+    outputs,
     ourIns: [],
     ourOuts: [],
     ourAmount: '0'

package/lib/common/utxobased/keymanager/keymanager.js

@@ -233,6 +233,7 @@ var VerifyAddressEnum; (function (VerifyAddressEnum) {
 
 
 
+
 
 
  function bip43PurposeNumberToTypeEnum(
@@ -829,7 +830,7 @@ const wifToPrivateKeyInternal = (
 } exports.signMessageBase64 = signMessageBase64;
 
  function makeTx(args) {
-  const { log } = args
+  const { log, outputSort } = args
   let sequence = 0xffffffff
   if (args.setRBF) {
     sequence -= 2
@@ -920,7 +921,7 @@ const wifToPrivateKeyInternal = (
       })
     }
 
-    if (target.memo != null) {
+    if (target.memo != null && target.memo !== '') {
       const memoHex = Buffer.from(target.memo, 'utf8').toString('hex')
       // check if hex string is within 80 bytes
       const validatedMemo = _utils.validateMemo.call(void 0, memoHex)
@@ -962,7 +963,14 @@ const wifToPrivateKeyInternal = (
   }
 
   const sortedInputs = _bip69.sortInputs.call(void 0, result.inputs)
-  const sortedOutputs = _bip69.sortOutputs.call(void 0, result.outputs)
+  const sortedOutputs = (() => {
+    switch (outputSort) {
+      case 'bip69':
+        return _bip69.sortOutputs.call(void 0, result.outputs)
+      case 'targets':
+        return result.outputs
+    }
+  })()
 
   const psbt = new bitcoin.Psbt()
   try {

package/lib/src/common/utxobased/keymanager/keymanager.d.ts

@@ -133,6 +133,7 @@ export interface MakeTxArgs {
     freshChangeAddress: string;
     subtractFee?: boolean;
     log?: EdgeLog;
+    outputSort: 'bip69' | 'targets';
 }
 export interface MakeTxTarget {
     address?: string;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "edge-currency-plugins",
-  "version": "1.1.4",
+  "version": "1.2.0",
   "description": "Edge currency plugins to fetch and process wallet accounts, balances and transactions.",
   "repository": "https://github.com/EdgeApp/edge-currency-plugins",
   "license": "MIT",