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.

Sign up for GitHub

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

Jump to bottom

Add a toString implementation for TransactionSimulatorResult #5942

Closed
macfarla opened this issue Sep 26, 2023 · 2 comments · Fixed by #5957
Closed

Add a toString implementation for TransactionSimulatorResult #5942

macfarla opened this issue Sep 26, 2023 · 2 comments · Fixed by #5957
Assignees
@Shritesh99
Labels
good first issue Good for newcomers snack Smaller coding task - less than a day for an experienced dev ux

Comments

@macfarla
Copy link
Contributor

Add a toString implementation for TransactionSimulatorResult

Default toString(): TransactionSimulatorResult inherits toString() from Object, and so is not suitable for printing.

see https://github.com/hyperledger/besu/security/code-scanning/1029
@macfarla macfarla added good first issue Good for newcomers snack Smaller coding task - less than a day for an experienced dev ux labels Sep 26, 2023
@macfarla macfarla mentioned this issue Sep 26, 2023
Merged
@Shritesh99
Copy link
Contributor

Hi, can I work on the issue?

@macfarla
Copy link
Contributor Author

Hi, can I work on the issue?

Go ahead @Shritesh99 - assigned the issue to you!

This was referenced Sep 27, 2023
Closed
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Shritesh99 Shritesh99

Labels
good first issue Good for newcomers snack Smaller coding task - less than a day for an experienced dev ux
Projects
None yet
Milestone
No milestone
2 participants
@macfarla @Shritesh99