Skip to content
This repository has been archived by the owner on Apr 4, 2024. It is now read-only.

Problem: fix eth log override #343 #371

Merged
merged 1 commit into from
Aug 19, 2021

Conversation

leejw51crypto
Copy link
Contributor

Solution: override with callback function

@leejw51crypto
Copy link
Contributor Author

leejw51crypto commented Jul 27, 2021

evm logs are printed only with debug option,
if it is printed in production, printing logs itself will dramatically slow down the performance.

@leejw51crypto
Copy link
Contributor Author

image

@fedekunze
Copy link
Contributor

thanks @leejw51crypto, can you fix the conflicts?

@leejw51crypto
Copy link
Contributor Author

leejw51crypto commented Aug 3, 2021

ok, i'll fix it

@leejw51crypto
Copy link
Contributor Author

rebase done

remove redundant log

tidy up
@fedekunze fedekunze merged commit c9fc183 into evmos:fedekunze/logger Aug 19, 2021
fedekunze added a commit that referenced this pull request Aug 19, 2021
* Problem: verbose logs display with FATAL option (fix #320)
add my script

increase amount for metamask

add run

amount ok

hide log

show info

my logger

hook log

revive eth log

tidy up

use suplog

log replace ok

removed suplog

tidy up

tidy up

fix compile

remove sh

tidy up

tidy up

* logger handler

* fix

* fix eth log override (#371)

remove redundant log

tidy up

* log test

* c++

Co-authored-by: jongwhan lee <jonghwan@crypto.com>
Co-authored-by: Jongwhan Lee <51560997+leejw51crypto@users.noreply.github.com>
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