-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
Issue with method call result with msg.sender and Geth 1.9.12 #3456
Comments
@Aniket-Engg Thanks for reporting, will investigate... |
@Aniket-Engg Copying over a summary of the discussion in ethereumjs gitter... @ryanio identified the cause of this as a Geth 1.9.12 breaking change. From: https://github.com/ethereum/go-ethereum/releases/tag/v1.9.12
The question now is whether or not Web3 should (try to) set an account for |
IMHO, Web3 can explain the situation in the documentation and recommend to explicitly define a With time, when users are aware about it, default support can be deprecated and removed. |
This also affects any Re-opening for visibility because it looks like Geth just published this change to their docker In our unit tests, when calling
|
Also happens to me
Specifying all the parameters: |
Is there any update on this? |
@GregTheGreek there were some notes added to the docs in #3467. a similar note could be added for estimateGas. |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed in 7 days if no further activity occurs. Thank you for your contributions. If you believe this was a mistake, please comment. |
Sample contract
Expected behavior
Actual balance of caller
Actual behavior
115792089237316195423570985008687907853269984665640564039457581007913129639935
Steps to reproduce the behavior
_getMyBalance
methodRelated Issues on different repos
https://github.com/ethereum/remix/issues/1430
ethereum/solidity#8545
Versions
web3js: 1.2.6
The text was updated successfully, but these errors were encountered: