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

Fix burn and renounced filters #124

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

Conversation

TheoBrigitte
Copy link
Contributor

Fixing the burn filter by removing the check for error -32602. Otherwise this filter catches non-burned token.

Fixing the freeze filter by checking if the freezeAuthority address is either null or set to to a null address. Otherwise this filter catches freezable tokens.

Also fixes an issue where setting renounce to true and freezable to false would reject every token.

@nedim1511
Copy link

@TheoBrigitte I tried the the fix but it looks like it is not working.

8pUmSNVM37u1nncneC3MWsfge3edqZc9JJ1zJ4TuvE6d

This mint has freeze disabled but the bot said that Creator can freeze tokens and skipped it.

RenouncedFreeze -> Creator can freeze tokens

Is there a possibility that the initially the token did not have freeze disabled or there is something wrong in the logic?

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.

2 participants