Content(add): smart contract interaction security page under wallet-security#377
Conversation
|
@artemisclaw82 is attempting to deploy a commit to the Security Alliance Team on Vercel. A member of the Team first needs to authorize it. |
f4a69a8 to
6b81fa6
Compare
|
Disclosure: This content was written by Artemis, an AI assistant operated by @DicksonWu654. It has not yet been reviewed by a human. Dickson will review and revise before marking as ready for review. Keeping as draft until that review is complete. |
docs/pages/wallet-security/smart-contract-interaction-security.mdx
Outdated
Show resolved
Hide resolved
6b81fa6 to
533dcf6
Compare
Sidebar Configuration ReminderDocumentation files update: New in this push:
Please ensure that:
See Contributing Guide – Sidebar & Navigation for more details. This is an automated reminder. If this PR doesn't need sidebar changes, you can ignore this message. |
docs/pages/wallet-security/smart-contract-interaction-security.mdx
Outdated
Show resolved
Hide resolved
docs/pages/wallet-security/smart-contract-interaction-security.mdx
Outdated
Show resolved
Hide resolved
docs/pages/wallet-security/smart-contract-interaction-security.mdx
Outdated
Show resolved
Hide resolved
docs/pages/wallet-security/smart-contract-interaction-security.mdx
Outdated
Show resolved
Hide resolved
docs/pages/wallet-security/smart-contract-interaction-security.mdx
Outdated
Show resolved
Hide resolved
docs/pages/wallet-security/smart-contract-interaction-security.mdx
Outdated
Show resolved
Hide resolved
artemisclaw82
left a comment
There was a problem hiding this comment.
Addressed all comments:
- QuillAudits contributor added to
contributors.jsonwith full profile (avatar, GitHub, Twitter, website, description) - Removed "— topics not covered elsewhere in the framework"
- Address poisoning expanded: fake tokens/NFTs, explorers, Safe UIs, don't copy from recent activity
- Clipboard malware now includes remediation: stop transacting, move funds from clean device, rotate credentials
- Multi-hop routes clarified: aggregators routing through unintended intermediary pools — this is a real DEX concern with aggregators like 1inch/Paraswap
- Ice phishing is real — coined by Microsoft in Feb 2022. Added citation.
- Permit signature phishing merged into the EIP-2612 section above (was redundant with it)
|
Fixed — reverted the contributors.json formatting changes. Now only adds the quillaudits entry without reformatting any existing entries. |
|
Fixed — restored the 4 deleted contributors (gunnim, madjin, monperrus, smagdali). They were lost when I reset to develop which did not have them yet. Now using the branch prior state as base, with only the quillaudits entry added. |
…ty-alliance#332) - New page: docs/pages/wallet-security/smart-contract-interaction-security.mdx - Add QuillAudits contributor to contributors.json - Add sidebar entry with dev: true Rebased cleanly on develop — no unrelated contributor changes.
ff0e800 to
c681b13
Compare
DicksonWu654
left a comment
There was a problem hiding this comment.
lgtm now - please undraft
built with Refined Cloudflare Pages Action⚡ Cloudflare Pages Deployment
|
|
@mattaereal ready to go! Thanks @DicksonWu654!! |
Summary
Adds a new page at
docs/pages/wallet-security/smart-contract-interaction-security.mdxcovering security practices for the interaction layer between having a secure wallet and signing a transaction. Updates sidebar navigation.Closes #332
What's included
approve()risks, exact vs unlimited amounts,permit()/EIP-2612 dangers, Revoke.cashFiles changed
docs/pages/wallet-security/smart-contract-interaction-security.mdx— New pagevocs.config.ts— Added Smart Contract Interaction Security to Wallet Security sidebar (dev: true)Design decisions
Template compliance
dev: true