-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
bump geth; replace common.Address.Hash() calls; restore standard log #11806
Conversation
I see that you haven't updated any README files. Would it make sense to do so? |
func init() { | ||
// hack to undo geth's disruption of the std default logger | ||
// remove with geth v1.13.10 | ||
slog.SetDefault(slog.New(slog.NewTextHandler(os.Stderr, nil))) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Geth is registering a discard logger from an init
that we have to replace to restore standard logs for goose, etc.: https://github.com/ethereum/go-ethereum/blob/38926cc6e507ba622275de789800e74215117dec/log/root.go#L13-L14
SonarQube Quality Gate |
I picked out some changes from #11799 to narrow down the issue over there.