Compare commits
1 Commits
feature/sh
...
fix/55-swa
| Author | SHA1 | Date | |
|---|---|---|---|
|
|
868f7a6ff8 |
35
README.md
35
README.md
@@ -213,22 +213,6 @@ create an address with the same visible characters and trick the user into
|
|||||||
sending funds to it. Showing the complete identifier defeats this class of
|
sending funds to it. Showing the complete identifier defeats this class of
|
||||||
attack.
|
attack.
|
||||||
|
|
||||||
#### Clipboard Policy
|
|
||||||
|
|
||||||
AutistMask never clears or overwrites the user's clipboard. When sensitive data
|
|
||||||
such as a private key is copied, it is the user's responsibility to manage their
|
|
||||||
clipboard afterwards. We deliberately avoid auto-clearing the clipboard for two
|
|
||||||
reasons:
|
|
||||||
|
|
||||||
1. **User expectations**: silently modifying the clipboard violates the
|
|
||||||
principle of least surprise. The user initiated the copy and knows the
|
|
||||||
content is sensitive.
|
|
||||||
2. **Data safety**: the user may have copied something else important in the
|
|
||||||
intervening time. A timed clipboard clear would destroy that unrelated data.
|
|
||||||
|
|
||||||
The warning shown before revealing a private key makes it clear that the key is
|
|
||||||
sensitive and that clipboard management is the user's responsibility.
|
|
||||||
|
|
||||||
#### Data Model
|
#### Data Model
|
||||||
|
|
||||||
The core hierarchy is **Wallets → Addresses**:
|
The core hierarchy is **Wallets → Addresses**:
|
||||||
@@ -332,34 +316,15 @@ transitions.
|
|||||||
- Balance list: ETH + tracked ERC-20 tokens (4 decimal places, USD inline).
|
- Balance list: ETH + tracked ERC-20 tokens (4 decimal places, USD inline).
|
||||||
Each balance row is clickable → **AddressToken**
|
Each balance row is clickable → **AddressToken**
|
||||||
- Send / Receive / + Token buttons
|
- Send / Receive / + Token buttons
|
||||||
- "Show private key" button
|
|
||||||
- Transaction list (with ENS resolution for counterparties)
|
- Transaction list (with ENS resolution for counterparties)
|
||||||
- **Transitions**:
|
- **Transitions**:
|
||||||
- Tap balance row → **AddressToken** (for that token)
|
- Tap balance row → **AddressToken** (for that token)
|
||||||
- "Send" → **Send**
|
- "Send" → **Send**
|
||||||
- "Receive" → **Receive**
|
- "Receive" → **Receive**
|
||||||
- "+ Token" → **AddToken**
|
- "+ Token" → **AddToken**
|
||||||
- "Show private key" → **ShowPrivateKey**
|
|
||||||
- Tap transaction row → **TransactionDetail**
|
- Tap transaction row → **TransactionDetail**
|
||||||
- "Back" → **Home**
|
- "Back" → **Home**
|
||||||
|
|
||||||
#### ShowPrivateKey
|
|
||||||
|
|
||||||
- **When**: User clicked "Show private key" on AddressDetail.
|
|
||||||
- **Elements**:
|
|
||||||
- "Back" button
|
|
||||||
- Title: "Display Private Key"
|
|
||||||
- Warning box (lock + money icons) explaining the key controls funds and
|
|
||||||
that the user is responsible for clipboard management
|
|
||||||
- Password input
|
|
||||||
- "Display Private Key" button (with lock + money icons)
|
|
||||||
- After reveal: private key in a read-only well (monospace, select-all),
|
|
||||||
Copy button, Done button
|
|
||||||
- **Transitions**:
|
|
||||||
- "Display Private Key" (correct password) → reveals key in-place
|
|
||||||
- "Copy" → copies key to clipboard
|
|
||||||
- "Done" / "Back" → **AddressDetail** (key cleared from DOM)
|
|
||||||
|
|
||||||
#### AddressToken
|
#### AddressToken
|
||||||
|
|
||||||
- **When**: User clicked a specific token balance on AddressDetail.
|
- **When**: User clicked a specific token balance on AddressDetail.
|
||||||
|
|||||||
@@ -307,15 +307,6 @@
|
|||||||
</button>
|
</button>
|
||||||
</div>
|
</div>
|
||||||
|
|
||||||
<div class="mb-3">
|
|
||||||
<button
|
|
||||||
id="btn-show-private-key"
|
|
||||||
class="border border-border px-2 py-1 hover:bg-fg hover:text-bg cursor-pointer text-xs"
|
|
||||||
>
|
|
||||||
🔒 Show private key
|
|
||||||
</button>
|
|
||||||
</div>
|
|
||||||
|
|
||||||
<!-- transactions -->
|
<!-- transactions -->
|
||||||
<div class="mt-3">
|
<div class="mt-3">
|
||||||
<div class="border-b border-border pb-1 mb-1">
|
<div class="border-b border-border pb-1 mb-1">
|
||||||
@@ -327,77 +318,6 @@
|
|||||||
</div>
|
</div>
|
||||||
</div>
|
</div>
|
||||||
|
|
||||||
<!-- ============ SHOW PRIVATE KEY ============ -->
|
|
||||||
<div id="view-show-private-key" class="view hidden">
|
|
||||||
<button
|
|
||||||
id="btn-show-pk-back"
|
|
||||||
class="border border-border px-2 py-1 hover:bg-fg hover:text-bg cursor-pointer mb-2"
|
|
||||||
>
|
|
||||||
< Back
|
|
||||||
</button>
|
|
||||||
<h2 class="font-bold mb-2">Display Private Key</h2>
|
|
||||||
|
|
||||||
<!-- password prompt section -->
|
|
||||||
<div id="show-pk-prompt">
|
|
||||||
<div
|
|
||||||
class="border border-border border-dashed p-3 mb-3 text-xs"
|
|
||||||
>
|
|
||||||
<p class="mb-1">
|
|
||||||
🔒💰 Your private key controls this
|
|
||||||
address and all its funds. Anyone who has it can
|
|
||||||
spend your tokens.
|
|
||||||
</p>
|
|
||||||
<p>
|
|
||||||
Do not share it. Do not paste it into websites. If
|
|
||||||
you copy it, you are responsible for clearing your
|
|
||||||
clipboard when you are done.
|
|
||||||
</p>
|
|
||||||
</div>
|
|
||||||
<div class="mb-2">
|
|
||||||
<label class="block mb-1">Password</label>
|
|
||||||
<input
|
|
||||||
type="password"
|
|
||||||
id="show-pk-password"
|
|
||||||
class="border border-border p-1 w-full font-mono text-sm bg-bg text-fg"
|
|
||||||
placeholder="Enter your password"
|
|
||||||
/>
|
|
||||||
</div>
|
|
||||||
<div
|
|
||||||
id="show-pk-error"
|
|
||||||
class="text-xs mb-2 border border-border border-dashed p-1 hidden"
|
|
||||||
></div>
|
|
||||||
<button
|
|
||||||
id="btn-show-pk-reveal"
|
|
||||||
class="border border-border px-2 py-1 hover:bg-fg hover:text-bg cursor-pointer"
|
|
||||||
>
|
|
||||||
🔒💰 Display Private Key
|
|
||||||
</button>
|
|
||||||
</div>
|
|
||||||
|
|
||||||
<!-- revealed key section -->
|
|
||||||
<div id="show-pk-key-well" class="hidden">
|
|
||||||
<div
|
|
||||||
class="bg-well p-3 mx-1 mb-3 break-all font-mono text-xs select-all"
|
|
||||||
>
|
|
||||||
<span id="show-pk-key-value"></span>
|
|
||||||
</div>
|
|
||||||
<div class="flex gap-2">
|
|
||||||
<button
|
|
||||||
id="btn-show-pk-copy"
|
|
||||||
class="border border-border px-2 py-1 hover:bg-fg hover:text-bg cursor-pointer"
|
|
||||||
>
|
|
||||||
Copy
|
|
||||||
</button>
|
|
||||||
<button
|
|
||||||
id="btn-show-pk-done"
|
|
||||||
class="border border-border px-2 py-1 hover:bg-fg hover:text-bg cursor-pointer"
|
|
||||||
>
|
|
||||||
Done
|
|
||||||
</button>
|
|
||||||
</div>
|
|
||||||
</div>
|
|
||||||
</div>
|
|
||||||
|
|
||||||
<!-- ============ ADDRESS-TOKEN DETAIL VIEW ============ -->
|
<!-- ============ ADDRESS-TOKEN DETAIL VIEW ============ -->
|
||||||
<div id="view-address-token" class="view hidden">
|
<div id="view-address-token" class="view hidden">
|
||||||
<button
|
<button
|
||||||
|
|||||||
@@ -19,7 +19,6 @@ const txStatus = require("./views/txStatus");
|
|||||||
const transactionDetail = require("./views/transactionDetail");
|
const transactionDetail = require("./views/transactionDetail");
|
||||||
const receive = require("./views/receive");
|
const receive = require("./views/receive");
|
||||||
const addToken = require("./views/addToken");
|
const addToken = require("./views/addToken");
|
||||||
const showPrivateKey = require("./views/showPrivateKey");
|
|
||||||
const settings = require("./views/settings");
|
const settings = require("./views/settings");
|
||||||
const settingsAddToken = require("./views/settingsAddToken");
|
const settingsAddToken = require("./views/settingsAddToken");
|
||||||
const approval = require("./views/approval");
|
const approval = require("./views/approval");
|
||||||
@@ -57,7 +56,6 @@ const ctx = {
|
|||||||
showAddWalletView: () => addWallet.show(),
|
showAddWalletView: () => addWallet.show(),
|
||||||
showImportKeyView: () => importKey.show(),
|
showImportKeyView: () => importKey.show(),
|
||||||
showAddressDetail: () => addressDetail.show(),
|
showAddressDetail: () => addressDetail.show(),
|
||||||
showPrivateKey: () => showPrivateKey.show(),
|
|
||||||
showAddressToken: () => addressToken.show(),
|
showAddressToken: () => addressToken.show(),
|
||||||
showAddTokenView: () => addToken.show(),
|
showAddTokenView: () => addToken.show(),
|
||||||
showConfirmTx: (txInfo) => confirmTx.show(txInfo),
|
showConfirmTx: (txInfo) => confirmTx.show(txInfo),
|
||||||
@@ -214,7 +212,6 @@ async function init() {
|
|||||||
importKey.init(ctx);
|
importKey.init(ctx);
|
||||||
home.init(ctx);
|
home.init(ctx);
|
||||||
addressDetail.init(ctx);
|
addressDetail.init(ctx);
|
||||||
showPrivateKey.init(ctx);
|
|
||||||
addressToken.init(ctx);
|
addressToken.init(ctx);
|
||||||
send.init(ctx);
|
send.init(ctx);
|
||||||
confirmTx.init(ctx);
|
confirmTx.init(ctx);
|
||||||
|
|||||||
@@ -187,7 +187,9 @@ function renderTransactions(txs) {
|
|||||||
let i = 0;
|
let i = 0;
|
||||||
for (const tx of txs) {
|
for (const tx of txs) {
|
||||||
const counterparty =
|
const counterparty =
|
||||||
tx.direction === "sent" || tx.direction === "contract"
|
tx.direction === "contract"
|
||||||
|
? tx.from
|
||||||
|
: tx.direction === "sent"
|
||||||
? tx.to
|
? tx.to
|
||||||
: tx.from;
|
: tx.from;
|
||||||
const ensName = ensNameMap.get(counterparty) || null;
|
const ensName = ensNameMap.get(counterparty) || null;
|
||||||
@@ -261,10 +263,6 @@ function init(_ctx) {
|
|||||||
});
|
});
|
||||||
|
|
||||||
$("btn-add-token").addEventListener("click", ctx.showAddTokenView);
|
$("btn-add-token").addEventListener("click", ctx.showAddTokenView);
|
||||||
|
|
||||||
$("btn-show-private-key").addEventListener("click", () => {
|
|
||||||
ctx.showPrivateKey();
|
|
||||||
});
|
|
||||||
}
|
}
|
||||||
|
|
||||||
module.exports = { init, show };
|
module.exports = { init, show };
|
||||||
|
|||||||
@@ -16,7 +16,6 @@ const VIEWS = [
|
|||||||
"import-key",
|
"import-key",
|
||||||
"main",
|
"main",
|
||||||
"address",
|
"address",
|
||||||
"show-private-key",
|
|
||||||
"address-token",
|
"address-token",
|
||||||
"send",
|
"send",
|
||||||
"confirm-tx",
|
"confirm-tx",
|
||||||
|
|||||||
@@ -1,79 +0,0 @@
|
|||||||
const { $, showView, showFlash, showError, hideError } = require("./helpers");
|
|
||||||
const { state } = require("../../shared/state");
|
|
||||||
const { decryptWithPassword } = require("../../shared/vault");
|
|
||||||
const { getPrivateKeyForAddress } = require("../../shared/wallet");
|
|
||||||
|
|
||||||
let ctx;
|
|
||||||
let revealed = false;
|
|
||||||
|
|
||||||
function show() {
|
|
||||||
revealed = false;
|
|
||||||
$("show-pk-password").value = "";
|
|
||||||
$("show-pk-key-well").classList.add("hidden");
|
|
||||||
$("show-pk-key-value").textContent = "";
|
|
||||||
$("show-pk-prompt").classList.remove("hidden");
|
|
||||||
hideError("show-pk-error");
|
|
||||||
showView("show-private-key");
|
|
||||||
}
|
|
||||||
|
|
||||||
function init(_ctx) {
|
|
||||||
ctx = _ctx;
|
|
||||||
|
|
||||||
$("btn-show-pk-back").addEventListener("click", () => {
|
|
||||||
clearKey();
|
|
||||||
ctx.showAddressDetail();
|
|
||||||
});
|
|
||||||
|
|
||||||
$("btn-show-pk-reveal").addEventListener("click", async () => {
|
|
||||||
const pw = $("show-pk-password").value;
|
|
||||||
if (!pw) {
|
|
||||||
showError("show-pk-error", "Please enter your password.");
|
|
||||||
return;
|
|
||||||
}
|
|
||||||
|
|
||||||
const wallet = state.wallets[state.selectedWallet];
|
|
||||||
let decryptedSecret;
|
|
||||||
try {
|
|
||||||
decryptedSecret = await decryptWithPassword(
|
|
||||||
wallet.encryptedSecret,
|
|
||||||
pw,
|
|
||||||
);
|
|
||||||
} catch (_e) {
|
|
||||||
showError("show-pk-error", "Wrong password.");
|
|
||||||
return;
|
|
||||||
}
|
|
||||||
|
|
||||||
const privateKey = getPrivateKeyForAddress(
|
|
||||||
wallet,
|
|
||||||
state.selectedAddress,
|
|
||||||
decryptedSecret,
|
|
||||||
);
|
|
||||||
|
|
||||||
revealed = true;
|
|
||||||
$("show-pk-prompt").classList.add("hidden");
|
|
||||||
$("show-pk-key-well").classList.remove("hidden");
|
|
||||||
$("show-pk-key-value").textContent = privateKey;
|
|
||||||
hideError("show-pk-error");
|
|
||||||
});
|
|
||||||
|
|
||||||
$("btn-show-pk-copy").addEventListener("click", () => {
|
|
||||||
const key = $("show-pk-key-value").textContent;
|
|
||||||
if (key) {
|
|
||||||
navigator.clipboard.writeText(key);
|
|
||||||
showFlash("Copied!");
|
|
||||||
}
|
|
||||||
});
|
|
||||||
|
|
||||||
$("btn-show-pk-done").addEventListener("click", () => {
|
|
||||||
clearKey();
|
|
||||||
ctx.showAddressDetail();
|
|
||||||
});
|
|
||||||
}
|
|
||||||
|
|
||||||
function clearKey() {
|
|
||||||
revealed = false;
|
|
||||||
$("show-pk-key-value").textContent = "";
|
|
||||||
$("show-pk-password").value = "";
|
|
||||||
}
|
|
||||||
|
|
||||||
module.exports = { init, show };
|
|
||||||
@@ -41,18 +41,6 @@ function getSignerForAddress(walletData, addrIndex, decryptedSecret) {
|
|||||||
return new Wallet(decryptedSecret);
|
return new Wallet(decryptedSecret);
|
||||||
}
|
}
|
||||||
|
|
||||||
function getPrivateKeyForAddress(walletData, addrIndex, decryptedSecret) {
|
|
||||||
if (walletData.type === "hd") {
|
|
||||||
const node = HDNodeWallet.fromPhrase(
|
|
||||||
decryptedSecret,
|
|
||||||
"",
|
|
||||||
BIP44_ETH_PATH,
|
|
||||||
);
|
|
||||||
return node.deriveChild(addrIndex).privateKey;
|
|
||||||
}
|
|
||||||
return decryptedSecret;
|
|
||||||
}
|
|
||||||
|
|
||||||
function isValidMnemonic(mnemonic) {
|
function isValidMnemonic(mnemonic) {
|
||||||
return Mnemonic.isValidMnemonic(mnemonic);
|
return Mnemonic.isValidMnemonic(mnemonic);
|
||||||
}
|
}
|
||||||
@@ -63,6 +51,5 @@ module.exports = {
|
|||||||
hdWalletFromMnemonic,
|
hdWalletFromMnemonic,
|
||||||
addressFromPrivateKey,
|
addressFromPrivateKey,
|
||||||
getSignerForAddress,
|
getSignerForAddress,
|
||||||
getPrivateKeyForAddress,
|
|
||||||
isValidMnemonic,
|
isValidMnemonic,
|
||||||
};
|
};
|
||||||
|
|||||||
Reference in New Issue
Block a user