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.

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

fix(rpc): align block number input behaviour for eth_getProof (backport: #1772) #233

Conversation

mmsqe
Copy link
Collaborator

@mmsqe mmsqe commented Mar 27, 2023

for more info: evmos#1722

Description


For contributor use:

  • Targeted PR against correct branch (see CONTRIBUTING.md)
  • Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
  • Code follows the module structure standards.
  • Wrote unit and integration tests
  • Updated relevant documentation (docs/) or specification (x/<module>/spec/)
  • Added relevant godoc comments.
  • Added a relevant changelog entry to the Unreleased section in CHANGELOG.md
  • Re-reviewed Files changed in the Github PR explorer

For admin use:

  • Added appropriate labels to PR (ex. WIP, R4R, docs, etc)
  • Reviewers assigned
  • Squashed all commits, uses message "Merge pull request #XYZ: [title]" (coding standards)

@github-advanced-security
Copy link

You have successfully added a new Semgrep configuration .github/workflows/semgrep.yml:semgrep. As part of the setup process, we have scanned this repository and found 41 existing alerts. Please check the repository Security tab to see all alerts.

@github-advanced-security
Copy link

You have successfully added a new gosec configuration .github/workflows/security.yml:Gosec. As part of the setup process, we have scanned this repository and found 141 existing alerts. Please check the repository Security tab to see all alerts.

@codecov
Copy link

codecov bot commented Mar 27, 2023

Codecov Report

Merging #233 (cb1c8a1) into release/v0.20.x-cronos (cb741e1) will decrease coverage by 0.08%.
The diff coverage is 0.00%.

Impacted file tree graph

@@                    Coverage Diff                     @@
##           release/v0.20.x-cronos     #233      +/-   ##
==========================================================
- Coverage                   57.20%   57.12%   -0.08%     
==========================================================
  Files                         108      108              
  Lines                       10149    10163      +14     
==========================================================
  Hits                         5806     5806              
- Misses                       4096     4110      +14     
  Partials                      247      247              
Impacted Files Coverage Δ
rpc/backend/call_tx.go 0.00% <0.00%> (ø)
x/evm/keeper/grpc_query.go 86.98% <0.00%> (-0.60%) ⬇️

@mmsqe mmsqe requested review from yihuang and calvinaco March 27, 2023 02:47
@yihuang yihuang merged commit 4ae22fe into crypto-org-chain:release/v0.20.x-cronos Apr 3, 2023
yihuang added a commit to yihuang/cronos that referenced this pull request Apr 3, 2023
Solution:
- update ethermint to include two bug fixes
  - crypto-org-chain/ethermint#234
  - crypto-org-chain/ethermint#233
- update cosmos-sdk to include one bug fix
  - cosmos/cosmos-sdk#15667
yihuang added a commit to yihuang/cronos that referenced this pull request Apr 3, 2023
Solution:
- update ethermint to include two bug fixes
  - crypto-org-chain/ethermint#234
  - crypto-org-chain/ethermint#233
- update cosmos-sdk to include one bug fix
  - cosmos/cosmos-sdk#15667
yihuang added a commit to crypto-org-chain/cronos that referenced this pull request Apr 4, 2023
* Problem: recent bug fixes not included

Solution:
- update ethermint to include two bug fixes
  - crypto-org-chain/ethermint#234
  - crypto-org-chain/ethermint#233
- update cosmos-sdk to include one bug fix
  - cosmos/cosmos-sdk#15667

* Update CHANGELOG.md

Signed-off-by: yihuang <huang@crypto.com>

* debug lint

* fix linter

---------

Signed-off-by: yihuang <huang@crypto.com>
mmsqe added a commit that referenced this pull request Jun 14, 2023
…rt: evmos#1772) (#233)

* add estimate_gas test

* add ret and vm_error for revert reason

* compare with eth

* test revertWithoutMsg

* align revertWithoutMsg response for eth_estimateGas

* align revertWithoutMsg response for eth_call

* add change doc

* separate contract to avoid gas change

* update mod

* fix resolve
mmsqe added a commit that referenced this pull request Jun 14, 2023
…rt: evmos#1722) (#270)

* fix(rpc): align block number input behaviour for eth_getProof (backport: evmos#1772) (#233)

* add estimate_gas test

* add ret and vm_error for revert reason

* compare with eth

* test revertWithoutMsg

* align revertWithoutMsg response for eth_estimateGas

* align revertWithoutMsg response for eth_call

* add change doc

* separate contract to avoid gas change

* update mod

* fix resolve

* fix doc
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants