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

Add utf-8 decoding for revert reason data #4791

Merged
merged 1 commit into from
Oct 29, 2021

Conversation

nikitosing
Copy link
Member

@nikitosing nikitosing commented Oct 20, 2021

Motivation

  • In some cases it is possible to decode revert reason without any ABI:

image

Changelog

Enhancements

  • Added utf-8 decoding for revert reason data

Bug Fixes

  • Fixed bug: raw revert data disappears after selecting "UTF-8" at the and of block with tx info

image

Checklist for your Pull Request (PR)

@nikitosing nikitosing force-pushed the np-add-utf-decoding-for-revert-reason branch 2 times, most recently from 05bce87 to 0e292ac Compare October 20, 2021 14:19
@nikitosing nikitosing force-pushed the np-add-utf-decoding-for-revert-reason branch from fdd93c1 to 62ab043 Compare October 27, 2021 09:12
@nikitosing nikitosing force-pushed the np-add-utf-decoding-for-revert-reason branch from 62ab043 to dae0ed5 Compare October 27, 2021 09:16
@vbaranov vbaranov merged commit ba354df into master Oct 29, 2021
@vbaranov vbaranov deleted the np-add-utf-decoding-for-revert-reason branch October 29, 2021 12:02
sidhujag pushed a commit to syscoin/blockscout that referenced this pull request Oct 29, 2021
…g-for-revert-reason

Add utf-8 decoding for revert reason data
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.

None yet

2 participants