plebbit / plebbit-react

A GUI for plebbit
https://plebbitapp.eth.limo
GNU General Public License v2.0
35 stars 6 forks source link

New NFT signature requires timestamp #253

Closed estebanabaroa closed 1 year ago

estebanabaroa commented 1 year ago

The NFT signatures have changed to

  messageThatShouldBeSigned.domainSeparator = 'plebbit-author-avatar'
  messageThatShouldBeSigned.authorAddress = authorAddress
  messageThatShouldBeSigned.timestamp = nft.timestamp
  messageThatShouldBeSigned.tokenAddress =  nft.address
  messageThatShouldBeSigned.tokenId = String(nft.id)

which is documented here https://github.com/plebbit/plebbit-js/blob/master/docs/nft.md

The current signature creator doesnt have the updated data:

Screenshot 2023-04-16 at 04-47-57 plebbit

{"domainSeparator":"plebbit-author-avatar","tokenAddress":"0x890a2e81836e0e76e0f49995e6b51ca6ce6f39ed","tokenId":"105","authorAddress":"estebanabaroa.eth"}

It is missing timestamp and also the order is incorrect. authorAddress is now second after domainSeparator.