NFT burn токена или что не так?

Question

[Александр Герасимов]

Здравствуйте!
Не буду тянуть "резину" и перейду сразу к делу. Рассмотрев несколько разных Смарт-контрактах я был удивлен, что мало кто закладывает burn токенов или вовсе не работает это. "Из коробки" у меня тоже не получилось это сделать. Есть наипростейший Смарт-контракт:

pragma solidity ^0.8.0;

import "@openzeppelin/contracts/utils/Counters.sol";
import "@openzeppelin/contracts/token/ERC721/ERC721.sol";
import "@openzeppelin/contracts/token/ERC721/extensions/ERC721URIStorage.sol";

contract SFFContract is ERC721URIStorage {
    using Counters for Counters.Counter;
    Counters.Counter private _tokenIds;

    constructor() ERC721("SFF NFT", "SFFNFT") {}

    function mint(address recipient, string memory uri) public returns (uint256) {
        _tokenIds.increment();

        uint256 newItemId = _tokenIds.current();
        _mint(recipient, newItemId);
        _setTokenURI(newItemId, uri);

        return newItemId;
    }

    function burn(uint256 tokenId) public virtual {
        _burn(tokenId);
    }
}

Так вот, транзакция с сжиганием токена не проходит. Она ревертится и имеет следующую ошибку:
Warning! Error encountered during contract execution [stack underflow (1 <=> 8)]

Судя по ошибке, то где-то происходит переполнение, т.е. типа 0 - 1 вываливается на границу. Если бы я сам писал функционал, то другое дело, но я использую базовые функций, которые, по моим представлениям должны работать без всякой магии.

Кто сталкивался с подобным? Какое решение нашли?

Answer 1

[chtulhu]

https://github.com/OpenZeppelin/openzeppelin-contr...

Вы используете _burn не проверяя есть ли этот токен вообще, как это сделано в расширении
https://github.com/OpenZeppelin/openzeppelin-contr...

Советую вам не мудрить в данном случае и воспользоваться визардом erc721 от того же openzeppelin
https://docs.openzeppelin.com/contracts/4.x/wizard (там есть опция burnable)

Comments

[Александр Герасимов]

Спасибо, что обратили внимание. Воспользовался я Визардом. Получил следующее:

pragma solidity ^0.8.0;

import "@openzeppelin/contracts/token/ERC721/ERC721.sol";
import "@openzeppelin/contracts/token/ERC721/extensions/ERC721URIStorage.sol";
import "@openzeppelin/contracts/token/ERC721/extensions/ERC721Burnable.sol";
import "@openzeppelin/contracts/access/Ownable.sol";
import "@openzeppelin/contracts/utils/Counters.sol";

contract SFFContract is ERC721, ERC721URIStorage, ERC721Burnable, Ownable {
    using Counters for Counters.Counter;

    Counters.Counter private _tokenIdCounter;

    constructor() ERC721("SFF NFT", "SFFNFT") {}

    function safeMint(address to, string memory uri) public onlyOwner {
        uint256 tokenId = _tokenIdCounter.current();
        _tokenIdCounter.increment();
        _safeMint(to, tokenId);
        _setTokenURI(tokenId, uri);
    }

    // The following functions are overrides required by Solidity.

    function _burn(uint256 tokenId) internal override(ERC721, ERC721URIStorage) {
        super._burn(tokenId);
    }

    function tokenURI(uint256 tokenId)
    public
    view
    override(ERC721, ERC721URIStorage)
    returns (string memory)
    {
        return super.tokenURI(tokenId);
    }
}

Опять же, майнтить получается, URI получается. Пытаюсь сжечь токен и такая же ошибка. Чувство, что я что-то не верно все же делаю. Стоит отметить, что использую Alchemy's Mempool. Ошибка такая же:
Warning! Error encountered during contract execution [stack underflow (1 <=> 8)]

Если что, то вот сама транзакция на burn:
https://rinkeby.etherscan.io/tx/0x090e68006225fc5e...

Немного кода из web3:

...
    const tx = {
        'nonce': nonce,
        'gas': 1000000,
        'gasLimit': 2000000,
        'data': nftContract.methods.burn(tokenId).encodeABI()
    };

    console.log("Burning token #" + tokenId + " ...");

    const signPromise = web3.eth.accounts.signTransaction(tx, PRIVATE_KEY);
...

[chtulhu]

Александр Герасимов, вы не задеплоили контракт, прежде чем его вызывать. eth_getCode по адресу 0x005c9ddf8defed79f94cb94260882b25b7f33097 выдает 0x

Ту транзакцию, которую вы прислали, эзерскан расценивает как попытка создания контракта.

Т.е. вам нужно как минимум отправлять две транзакции:
1. Создание контракта
2. Вызов метода burn по адресу уже этого нового контракта

[Александр Герасимов]

chtulhu, Я не знаю где и почему eth_getCode выдает 0x.

Я скомпилировал контракт:

npx hardhat compile

Я задеплоил контракт:

npx hardhat --network rinkeby run scripts/deploy.js

Естественно проставив приватные ключи и прочее. Иначе бы вообще не получилось продвинуться дальше.

Я замайнил 3 токена (использовал web3 и детали опущу):

node scripts/mint.js

Эти 3 токена так же видны в контракте (транзакции видны). Ах, да, сам контракт вот:
https://rinkeby.etherscan.io/address/0x9C5DF4c11c3...

По контракту вы можете выйти на Creator и посмотреть все мои "потуги" по сжиганию. Они все имеют статус FAIL.

Контракту я не делал верификацию. Это ведь необязательное условие? Иначе майнтить можно, а сжигать нет - странная ситуация была бы.

[chtulhu]

Александр Герасимов, "Я задеплоил контракт:"

точнее будет сказать вы запустили скрипт, который должен делать деплой. Видимо, проблема только в том, что вы не указываете адрес контракта, когда конструируете транзакцию (поле to) и поэтому блокчейн это расценивает как попытку создать новый контракт
const tx = {
...

[Александр Герасимов]

chtulhu, когда я задеплоил контракт, то я получил его адрес. Который положил в .env-файл и использовал дальше. Например, этот же адрес контракта используется для чеканки, для получения tokenURI() и они отрабатывают корректно. Токены чеканятся, адрес токена возвращается правильный. Но вот сжечь токен по его tokenId не получается. В общем-то там используется такая же транзакция, как и при чеканке токена. Даже удивляюсь, почему на ровном месте затычка у меня.

А подскажите, где вы вызываете eth_getCode()? Что это за инструмент? Тоже поковыряюсь там, что-то полезное для себя еще открою.

[chtulhu]

Александр Герасимов, Рекомендую услышать меня, если хотите решить свою проблему )) Повторюсь, вы НЕ вызывает burn, вы создаете новый контракт т.к. ваша транзакция не содержит адрес контракта(поле to). Вы прислали мне свою транзакцию:

https://rinkeby.etherscan.io/tx/0x090e68006225fc5e...

там черным по белому написано:
[Contract 0x005c9ddf8defed79f94cb94260882b25b7f33097 Created] -- т.е. эта транзакция создает контракт хоть и битый, а не вызывает ваш уже созданный.

Вот ваша транзакция под капотом(нет поля to):
"hash" => "0x090e68006225fc5ef0bf81e6e839622272b3f6af5fd387eb75fcf53117bcee1e"
"blockHash" => "0x67a3e51c70c6d24652e756b8ba9356cdf0ba1cb9e02ae90d72d2d83287231862"
"blockNumber" => "0x9d83ec"
"from" => "0xe6f42ba58e22c3c3ae4ff7c56958b912112b2149"
"gas" => "0x1e8480"
"gasPrice" => "0x4a4c7ca3"
"input" => "0x42966c680000000000000000000000000000000000000000000000000000000000000001"
"nonce" => "0x6e"
"r" => "0xce6612dd1ec3138a895a987e0e206d18ee70b561e7c6f4da41fcfda013062a27"
"s" => "0x37d93674a77560ef8a0cb6fd1328d488b087854442526eb9503c3586c6c863c5"
"transactionIndex" => "0x2c"
"type" => "0x0"
"v" => "0x2b"
"value" => "0x0"

[Александр Герасимов]

chtulhu, мир не без добрых людей! Спасибо большое!
Действительно, прописал для транзакции поле to и удалось сжечь. Пойду причесывать и копаться с контрактом дальше.

[chtulhu]

Александр Герасимов, не за что! )) eth_getCode это один из методов JSON RPC API:

https://eth.wiki/json-rpc/API