aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
-rw-r--r--routes/bitcoin/conversionFee.js12
-rw-r--r--routes/bitcoin/receive.js6
-rw-r--r--routes/bitcoin/send.js35
-rw-r--r--routes/liquid/conversionFee.js12
-rw-r--r--routes/liquid/receive.js5
-rw-r--r--routes/liquid/send.js37
6 files changed, 64 insertions, 43 deletions
diff --git a/routes/bitcoin/conversionFee.js b/routes/bitcoin/conversionFee.js
new file mode 100644
index 00000000..8a6bfce6
--- /dev/null
+++ b/routes/bitcoin/conversionFee.js
@@ -0,0 +1,12 @@
+// Code for the conversion fee
+// This is in a separate file so that the same function can be used by ./send and ./receive
+
+const conversionFeeMultiplier = 0.01;
+export const conversionFeeReceiver = "coinosfees"; // account that receives the fees
+
+/**
+ * Returns the fee you have to pay for withdrawing 'amount' bitcoin.
+ */
+export function computeConversionFee(amount) {
+ return Math.floor(amount * conversionFeeMultiplier);
+}
diff --git a/routes/bitcoin/receive.js b/routes/bitcoin/receive.js
index 5bdbcb72..02deb58f 100644
--- a/routes/bitcoin/receive.js
+++ b/routes/bitcoin/receive.js
@@ -21,8 +21,7 @@ const network =
const queue = {};
const seen = [];
-// used to calculate how many credits to give
-const withdrawalFeeMultiplier = 0.01;
+import { computeConversionFee } from './conversionFee.js';
zmqRawTx.on("message", async (topic, message, sequence) => {
const hex = message.toString("hex");
@@ -195,7 +194,8 @@ setInterval(async () => {
{ pending: Math.min(account.pending, total) },
{ transaction }
);
- await account.increment({ fee_credits: Math.floor(total * withdrawalFeeMultiplier) }, { transaction });
+ // get the # of fee credits you would need to pay off this amount of bitcoin
+ await account.increment({ fee_credits: computeConversionFee(total) }, { transaction });
await account.reload({ transaction });
await p.save({ transaction });
diff --git a/routes/bitcoin/send.js b/routes/bitcoin/send.js
index a9c95b31..72d34317 100644
--- a/routes/bitcoin/send.js
+++ b/routes/bitcoin/send.js
@@ -1,6 +1,5 @@
const btc = config.liquid.btcasset;
-const withdrawalFeeMultiplier = 0.01; // 1% withdrawal fee
-const withdrawalFeeReceiver = "coinosfees"; // account that receives the fees
+import { computeConversionFee, conversionFeeReceiver } from './conversionFee.js';
module.exports = ah(async (req, res) => {
let { user } = req;
@@ -31,10 +30,10 @@ module.exports = ah(async (req, res) => {
total = total - change + fee;
let amount = total - fee;
- // get withdrawal fee
- // 'total' refers to the total before the withdrawal fee
+ // get conversion fee
+ // 'total' refers to the total before the conversion fee
// (i.e. the total bitcoin that leaves this server)
- let withdrawalFee = Math.floor(amount * withdrawalFeeMultiplier);
+ let conversionFee = computeConversionFee(amount);
try {
// withdraw bitcoin
@@ -62,25 +61,25 @@ module.exports = ah(async (req, res) => {
if (total > account.balance) {
l.error("amount exceeds balance", amount, fee, account.balance);
throw new Error("low balance");
- } else if (total + withdrawalFee > account.balance) {
- l.error("total (after withdrawal fee) exceeds balance", amount, fee, account.balance);
- throw new Error("low balance (after withdrawal fee)");
+ } else if (total + conversionFee > account.balance) {
+ l.error("total (after conversion fee) exceeds balance", amount, fee, account.balance);
+ throw new Error("low balance (after conversion fee)");
}
// use user's credits to reduce fee, if available
- let withdrawalFeeDeduction = Math.min(account.fee_credits, withdrawalFee);
- if (withdrawalFeeDeduction) {
- await account.decrement({ fee_credits: withdrawalFeeDeduction }, { transaction });
+ let conversionFeeDeduction = Math.min(account.fee_credits, conversionFee);
+ if (conversionFeeDeduction) {
+ await account.decrement({ fee_credits: conversionFeeDeduction }, { transaction });
await account.reload({ transaction });
- withdrawalFee -= withdrawalFeeDeduction;
+ conversionFee -= conversionFeeDeduction;
}
- await account.decrement({ balance: (total + withdrawalFee) }, { transaction });
+ await account.decrement({ balance: (total + conversionFee) }, { transaction });
await account.reload({ transaction });
let receiverAccount = await db.Account.findOne({
where: {
- "$user.username$": withdrawalFeeReceiver
+ "$user.username$": conversionFeeReceiver
},
include: [
{
@@ -93,13 +92,13 @@ module.exports = ah(async (req, res) => {
});
let fee_payment_id = null;
- if (withdrawalFee) {
- await receiverAccount.increment({ balance: withdrawalFee }, { transaction });
+ if (conversionFee) {
+ await receiverAccount.increment({ balance: conversionFee }, { transaction });
await receiverAccount.reload({ transaction });
let fee_payment = await db.Payment.create({
- amount: withdrawalFee,
+ amount: conversionFee,
fee: 0,
- memo: "Bitcoin withdrawal fee",
+ memo: "Bitcoin conversion fee",
account_id: receiverAccount.id,
user_id: receiverAccount.user_id,
rate: app.get("rates")[receiverAccount.user.currency],
diff --git a/routes/liquid/conversionFee.js b/routes/liquid/conversionFee.js
new file mode 100644
index 00000000..36ba6f36
--- /dev/null
+++ b/routes/liquid/conversionFee.js
@@ -0,0 +1,12 @@
+// Code for the conversion fee
+// This is in a separate file so that the same function can be used by ./send and ./receive
+
+const conversionFeeMultiplier = 0.01;
+export const conversionFeeReceiver = "coinosfees"; // account that receives the fees
+
+/**
+ * Returns the fee you have to pay for withdrawing 'amount' L-BTC.
+ */
+export function computeConversionFee(amount) {
+ return Math.floor(amount * conversionFeeMultiplier);
+}
diff --git a/routes/liquid/receive.js b/routes/liquid/receive.js
index 9650e5b7..432619b3 100644
--- a/routes/liquid/receive.js
+++ b/routes/liquid/receive.js
@@ -13,8 +13,7 @@ const network =
config.liquid.network === "mainnet" ? "liquid" : config.liquid.network
];
-// used to calculate how many credits to give
-const withdrawalFeeMultiplier = 0.01;
+import { computeConversionFee } from './conversionFee.js';
const getAccount = async (params, transaction) => {
let account = await db.Account.findOne({
@@ -344,7 +343,7 @@ setInterval(async () => {
{ pending: Math.min(p.account.pending, total) },
{ transaction }
);
- await p.account.increment({ fee_credits: Math.floor(total * withdrawalFeeMultiplier) }, { transaction });
+ await p.account.increment({ fee_credits: computeConversionFee(total) }, { transaction });
await p.account.reload({ transaction });
await p.save({ transaction });
diff --git a/routes/liquid/send.js b/routes/liquid/send.js
index fcb05942..33bee9ed 100644
--- a/routes/liquid/send.js
+++ b/routes/liquid/send.js
@@ -1,8 +1,7 @@
const btc = config.liquid.btcasset;
const lcad = config.liquid.cadasset;
const { Transaction } = require("liquidjs-lib");
-const withdrawalFeeMultiplier = 0.01; // 1% withdrawal fee
-const withdrawalFeeReceiver = "coinosfees"; // 1% withdrawal fee
+import { computeConversionFee, conversionFeeReceiver } from './conversionFee.js';
sendLiquid = async ({ asset, amount, user, address, memo, tx, limit }) => {
try {
@@ -84,10 +83,10 @@ sendLiquid = async ({ asset, amount, user, address, memo, tx, limit }) => {
total += fee - covered;
}
- // get withdrawal fee
- // 'total' refers to the total before the withdrawal fee
+ // get conversion fee
+ // 'total' refers to the total before the conversion fee
// (i.e. the total liquid bitcoin that leaves this server)
- var withdrawalFee = Math.floor(amount * withdrawalFeeMultiplier);
+ var conversionFee = computeConversionFee(amount);
if (limit && total > limit + fee)
throw new Error("Tx amount exceeds authorized amount");
@@ -117,15 +116,15 @@ sendLiquid = async ({ asset, amount, user, address, memo, tx, limit }) => {
account.ticker === "BTC" ? "SAT" : account.ticker
}, have ${account.balance}`
);
- } else if (total + withdrawalFee > account.balance) {
- l.warn("amount plus withdrawal fee exceeds balance", {
+ } else if (total + conversionFee > account.balance) {
+ l.warn("amount plus conversion fee exceeds balance", {
total,
fee,
- withdrawalFee,
+ conversionFee,
balance: account.balance
});
throw new Error(
- `Insufficient funds, need ${total + withdrawalFee} ${
+ `Insufficient funds, need ${total + conversionFee} ${
account.ticker === "BTC" ? "SAT" : account.ticker
}, have ${account.balance}`
);
@@ -133,19 +132,19 @@ sendLiquid = async ({ asset, amount, user, address, memo, tx, limit }) => {
// use user's credits to reduce fee, if available
console.log(account.fee_credits);
- let withdrawalFeeDeduction = Math.min(account.fee_credits, withdrawalFee);
- if (withdrawalFeeDeduction) {
- await account.decrement({ fee_credits: withdrawalFeeDeduction }, { transaction });
+ let conversionFeeDeduction = Math.min(account.fee_credits, conversionFee);
+ if (conversionFeeDeduction) {
+ await account.decrement({ fee_credits: conversionFeeDeduction }, { transaction });
await account.reload({ transaction });
- withdrawalFee -= withdrawalFeeDeduction;
+ conversionFee -= conversionFeeDeduction;
}
- await account.decrement({ balance: (total + withdrawalFee)}, { transaction });
+ await account.decrement({ balance: (total + conversionFee)}, { transaction });
await account.reload({ transaction });
let receiverAccount = await db.Account.findOne({
where: {
- "$user.username$": withdrawalFeeReceiver
+ "$user.username$": conversionFeeReceiver
},
include: [
{
@@ -158,13 +157,13 @@ sendLiquid = async ({ asset, amount, user, address, memo, tx, limit }) => {
});
let fee_payment_id = null;
- if (withdrawalFee) {
- await receiverAccount.increment({ balance: withdrawalFee }, { transaction });
+ if (conversionFee) {
+ await receiverAccount.increment({ balance: conversionFee }, { transaction });
await receiverAccount.reload({ transaction });
let fee_payment = {
- amount: withdrawalFee,
+ amount: conversionFee,
fee: 0,
- memo: "Liquid withdrawal fee",
+ memo: "Liquid conversion fee",
account_id: receiverAccount.id,
user_id: receiverAccount.user_id,
rate: app.get("rates")[receiverAccount.user.currency],