* Update proto make commands to use cosmos docker image and add notes for possible problems
* Apply make proto-all with new docker container
* Remove stale DOCKER_BUF variable
* Revert to using the tendermintdev/sdk-proto-gen docker image
* remove '@' in proto-lint and proto-check-breaking for consistency with other commands
* Remove unnecessary go get from protocgen.sh (only works after adding --network host to docker run)
* Add --network host to docker run for compatibility on linux
* use cosmos/proto-builder docker image for proto-format because clang-format is not installed on tendermintdev/sdk-proto-gen
* update swagger docs after recent additions to evm.proto in #1413
Co-authored-by: Tomas Guerra <54514587+GAtom22@users.noreply.github.com>
* fix: chain-id in grpc query is not initialized without abci event
Closes: #1404
Solution:
- pass the chain-id from caller.
* Update CHANGELOG.md
* only override if input is not empty
* add comment to chain id
* pass chain-id to state transition
* Update x/evm/keeper/grpc_query.go
* Apply suggestions from code review
* fix golang lint
* update gomod2nix.toml
* fix unit tests
* update gomod2nix
* api breaking changelog
* add unit tests, and fix TraceBlock by the way
* Update CHANGELOG.md
* test --grpc-only mode in integration tests
* remove tmp var
* Update tests/integration_tests/test_grpc_only.py
* Update x/evm/keeper/grpc_query_test.go
Co-authored-by: mmsqe <tqd0800210105@gmail.com>
* fix linters
* fix nil pointer in tests
* fix conflicts
* fix conflicts
* fixes
* fix lint
* fix unit test
Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com>
Co-authored-by: mmsqe <tqd0800210105@gmail.com>
Co-authored-by: Freddy Caceres <facs95@gmail.com>
* Delete local copy of third party proto files
* Update protocgen script and buf yaml files to mirror cosmos-sdk
* Update makefile commands for proto-gen and proto-swagger-gen to correctly use docker
* Commit changed .pb.go files after updating the protogen scripts
* Adjust grep in proto-tools-installer script to look for correct gogoproto replacement
* address reviews - remove unnecessary ignore in buf.yaml and cosmos-sdk download in the protocgen script
* remove proto-update-deps from makefile as we don't store local copies of third party protofiles anymore
* Add changelog entry
* Update protoc-swagger-gen.sh
* Remove third party queries from swagger-ui config (for now)
* fix integrations tests
* fix dead changelog links (markdown-link-check)
Co-authored-by: Freddy Caceres <facs95@gmail.com>
Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com>
* impr(evm): add check msg.Validate check to prevent 0 gas txs
* add changelog
* impr(evm): add integration tests for zero gas txs
* go mod tidy and proto swagger
* fix comment
* move changelog to state machine breaking
* add ReadHeaderTimeout`
* revert ReadHeaderTimeout config change
Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com>
Closes: #868
Solution:
- contract address in receipt should only be the newly created contract
- fix the order of operations in state transition
- compute contract address in `ApplyTransaction` on the fly
Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com>
* do binary search to estimate gas
Closes#268
- Also refactor ApplyMessage to be more reuseable
move binary search to rpc api side to have a clean context each try
remove EstimateGas grpc api
* extract BinSearch function and add unit test
* do estimateGas in grpc query
Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com>
