feat(solana): add support for Token-2022 tokens #15769
Conversation
MartinZemanik
requested review from
mroz22,
szymonlesisz,
martykan and
marekrjpolak
as code owners
| @@ -44,6 +44,7 @@ export default class BlockchainEstimateFee extends AbstractMethod<'blockchainEst | |||
| { name: 'to', type: 'string' }, | |||
| { name: 'txsize', type: 'number' }, | |||
| { name: 'isCreatingAccount', type: 'boolean' }, | |||
| { name: 'newTokenAccountProgramName', type: 'string' }, | |||
There was a problem hiding this comment.
Should I add validation that it's one of ['spl-token', 'spl-token-2022']?
There was a problem hiding this comment.
Hello @MartinZemanik, would be great
|
🚨 Potential security issues detected. Learn more about Socket for GitHub ↗︎ To accept the risk, merge this PR and you will not be notified again.
Next stepsWhat are unpopular packages?This package is not very popular. Unpopular packages may have less maintenance and contain other problems. Take a deeper look at the dependencyTake a moment to review the security alert above. Review the linked package source code to understand the potential risk. Ensure the package is not malicious before proceeding. If you're unsure how to proceed, reach out to your security team or ask the Socket team for help at support [AT] socket [DOT] dev. Remove the packageIf you happen to install a dependency that Socket reports as Known Malware you should immediately remove it and select a different dependency. For other alert types, you may may wish to investigate alternative packages or consider if there are other ways to mitigate the specific risk posed by the dependency. Mark a package as acceptable riskTo ignore an alert, reply with a comment starting with
|
|
We merged the pr from our branch. Thank you! Closing #15902 |
This PR adds support for Solana Token-2022 tokens. This means:
Screenshots:
Tokens list
Send token
Transaction history