Skip to content
This repository has been archived by the owner on Aug 2, 2022. It is now read-only.

Display assert to user when --verbose-http-errors not specified #6769

Merged
merged 9 commits into from
Feb 21, 2019

Conversation

heifner
Copy link
Contributor

@heifner heifner commented Feb 18, 2019

Change Description

  • Currently the contract's eosio_assert message is not reported to the user unless --verbose-http-errors is specified to nodeos. For example:
./cleos push action asserter procassert '[0, "hello"]' -p asserter
Error 3050003: eosio_assert_message assertion failure
  • This PR adds minimum log_message output so that eosio_assert messages are reported to the user even when --verbose-http-errors is not enabled. For example:
./cleos push action asserter procassert '[0, "hello"]' -p asserter
Error 3050003: eosio_assert_message assertion failure
Error Details:
assertion failure with message: hello

Consensus Changes

None

API Changes

None

Documentation Additions

  • Some examples may need to be updated to reflect that assert_message now contains the user assert message.

@heifner heifner changed the base branch from develop to release/1.6.x February 21, 2019 02:26
@heifner heifner changed the base branch from release/1.6.x to develop February 21, 2019 02:26
@arhag arhag merged commit 6307ff0 into develop Feb 21, 2019
@arhag arhag deleted the better-errors branch February 21, 2019 02:53
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants