Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update whitelist.yaml #1417

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Update whitelist.yaml #1417

wants to merge 2 commits into from

Conversation

woodydeck
Copy link

The dapp does not call a contract. It is a simple token send with a memo attached of a pump.fun token. The dapp reads Tx matching criteria of Tx directed at the receiving address and then ingests into our database as a message which is then printed on the wall.

Clarity in what a transaction does is a better strategy than blocking simple token sends. Whitelisting of tokens should be done at the user level, not with PRs. This is a poor strategy for security and MetaMask has failed with it, which I know is the intent of this move.

Now progress is stalled because of an arbitrary rule from a de facto gatekeeper which has had zero impact on the amount of fraud in the space.

The dapp does not call a contract. It is a simple token send with a memo attached of a pump.fun token. The dapp reads Tx matching criteria of Tx directed at the receiving address.
@woodydeck
Copy link
Author

Please see the issue: #1421

The whitelisting is needed to get around Blowfish's arbitrary censorship. Corresponding with [email protected] does nothing, they ignore everything. This is the responsibility of the Phantom project to correct. It cannot simply be passed off to what a reasonable person would regard as a malicious third party.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant