Compare commits
1 Commits
b799686cd4
...
feature/is
| Author | SHA1 | Date | |
|---|---|---|---|
|
|
7bd6b5bbdc |
@@ -577,19 +577,6 @@
|
|||||||
<div id="confirm-fee-amount" class="text-xs"></div>
|
<div id="confirm-fee-amount" class="text-xs"></div>
|
||||||
</div>
|
</div>
|
||||||
<div id="confirm-warnings" class="mb-2 hidden"></div>
|
<div id="confirm-warnings" class="mb-2 hidden"></div>
|
||||||
<div
|
|
||||||
id="confirm-recipient-warning"
|
|
||||||
class="mb-2"
|
|
||||||
style="visibility: hidden"
|
|
||||||
>
|
|
||||||
<div
|
|
||||||
class="border border-red-500 border-dashed p-2 text-xs font-bold text-red-500"
|
|
||||||
>
|
|
||||||
WARNING: The recipient address has ZERO transaction
|
|
||||||
history. This may indicate a fresh or unused address.
|
|
||||||
Double-check the address before sending.
|
|
||||||
</div>
|
|
||||||
</div>
|
|
||||||
<div
|
<div
|
||||||
id="confirm-errors"
|
id="confirm-errors"
|
||||||
class="mb-2 border border-border border-dashed p-2 hidden"
|
class="mb-2 border border-border border-dashed p-2 hidden"
|
||||||
|
|||||||
@@ -25,6 +25,7 @@ const { decryptWithPassword } = require("../../shared/vault");
|
|||||||
const { formatUsd, getPrice } = require("../../shared/prices");
|
const { formatUsd, getPrice } = require("../../shared/prices");
|
||||||
const { getProvider } = require("../../shared/balances");
|
const { getProvider } = require("../../shared/balances");
|
||||||
const { isScamAddress } = require("../../shared/scamlist");
|
const { isScamAddress } = require("../../shared/scamlist");
|
||||||
|
const { hasZeroTransactionHistory } = require("../../shared/transactions");
|
||||||
const { ERC20_ABI } = require("../../shared/constants");
|
const { ERC20_ABI } = require("../../shared/constants");
|
||||||
const { log } = require("../../shared/log");
|
const { log } = require("../../shared/log");
|
||||||
const makeBlockie = require("ethereum-blockies-base64");
|
const makeBlockie = require("ethereum-blockies-base64");
|
||||||
@@ -243,12 +244,6 @@ function show(txInfo) {
|
|||||||
state.viewData = { pendingTx: txInfo };
|
state.viewData = { pendingTx: txInfo };
|
||||||
showView("confirm-tx");
|
showView("confirm-tx");
|
||||||
|
|
||||||
// Reset recipient warning: reserve space (visibility:hidden) while
|
|
||||||
// the async check runs, preventing layout shift per README policy.
|
|
||||||
const recipientWarning = $("confirm-recipient-warning");
|
|
||||||
recipientWarning.style.display = "";
|
|
||||||
recipientWarning.style.visibility = "hidden";
|
|
||||||
|
|
||||||
estimateGas(txInfo);
|
estimateGas(txInfo);
|
||||||
checkRecipientHistory(txInfo);
|
checkRecipientHistory(txInfo);
|
||||||
}
|
}
|
||||||
@@ -294,31 +289,20 @@ async function estimateGas(txInfo) {
|
|||||||
}
|
}
|
||||||
|
|
||||||
async function checkRecipientHistory(txInfo) {
|
async function checkRecipientHistory(txInfo) {
|
||||||
const el = $("confirm-recipient-warning");
|
const isNew = await hasZeroTransactionHistory(
|
||||||
try {
|
txInfo.to,
|
||||||
const provider = getProvider(state.rpcUrl);
|
state.blockscoutUrl,
|
||||||
// Skip warning for contract addresses — they may legitimately
|
);
|
||||||
// have zero outgoing transactions (getTransactionCount returns
|
if (!isNew) return;
|
||||||
// the nonce, i.e. sent-tx count only).
|
|
||||||
const code = await provider.getCode(txInfo.to);
|
const warningsEl = $("confirm-warnings");
|
||||||
if (code && code !== "0x") {
|
const warningHtml =
|
||||||
// Contract address — hide the reserved space entirely
|
`<div class="border border-red-500 border-dashed p-2 mb-1 text-xs font-bold text-red-500">` +
|
||||||
el.style.display = "none";
|
`WARNING: This address has ZERO transaction history. ` +
|
||||||
return;
|
`It has never sent or received any funds. ` +
|
||||||
}
|
`Double-check the address before sending.</div>`;
|
||||||
const txCount = await provider.getTransactionCount(txInfo.to);
|
warningsEl.innerHTML = warningHtml + warningsEl.innerHTML;
|
||||||
if (txCount === 0) {
|
warningsEl.classList.remove("hidden");
|
||||||
el.style.visibility = "visible";
|
|
||||||
} else {
|
|
||||||
// Address has history — collapse the reserved space
|
|
||||||
el.style.display = "none";
|
|
||||||
}
|
|
||||||
} catch (e) {
|
|
||||||
log.errorf("recipient history check failed:", e.message);
|
|
||||||
// On error, collapse the reserved space rather than showing a
|
|
||||||
// false warning or leaving an empty gap
|
|
||||||
el.style.display = "none";
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
|
|
||||||
function init(ctx) {
|
function init(ctx) {
|
||||||
|
|||||||
@@ -251,4 +251,40 @@ function filterTransactions(txs, filters = {}) {
|
|||||||
return { transactions: filtered, newFraudContracts: newFraud };
|
return { transactions: filtered, newFraudContracts: newFraud };
|
||||||
}
|
}
|
||||||
|
|
||||||
module.exports = { fetchRecentTransactions, filterTransactions };
|
/**
|
||||||
|
* Check whether an address has any on-chain transaction history.
|
||||||
|
* Returns true if the address has zero normal transactions AND zero
|
||||||
|
* token transfers on the configured Blockscout instance.
|
||||||
|
* Returns false on network errors (fail-open: don't block sends).
|
||||||
|
*/
|
||||||
|
async function hasZeroTransactionHistory(address, blockscoutUrl) {
|
||||||
|
try {
|
||||||
|
const resp = await debugFetch(
|
||||||
|
blockscoutUrl + "/addresses/" + address + "/transactions?limit=1",
|
||||||
|
);
|
||||||
|
if (!resp.ok) return false;
|
||||||
|
const json = await resp.json();
|
||||||
|
if ((json.items || []).length > 0) return false;
|
||||||
|
|
||||||
|
// Also check token transfers — an address may have only received
|
||||||
|
// ERC-20 tokens without any native ETH transactions.
|
||||||
|
const ttResp = await debugFetch(
|
||||||
|
blockscoutUrl +
|
||||||
|
"/addresses/" +
|
||||||
|
address +
|
||||||
|
"/token-transfers?type=ERC-20&limit=1",
|
||||||
|
);
|
||||||
|
if (!ttResp.ok) return false;
|
||||||
|
const ttJson = await ttResp.json();
|
||||||
|
return (ttJson.items || []).length === 0;
|
||||||
|
} catch (e) {
|
||||||
|
log.errorf("hasZeroTransactionHistory check failed:", e.message);
|
||||||
|
return false;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
module.exports = {
|
||||||
|
fetchRecentTransactions,
|
||||||
|
filterTransactions,
|
||||||
|
hasZeroTransactionHistory,
|
||||||
|
};
|
||||||
|
|||||||
Reference in New Issue
Block a user