-
Notifications
You must be signed in to change notification settings - Fork 862
Backport release/v6.3: Adjusted RPC http requests to use POST instead of GET
#2687
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
Conversation
RPC calls for the EVM CLI were attempting to send methods via GET instead of POST, resulting in failed extraction of results from the associated RPC calls as none were being returned. Adjustments to `getChainId()`, `getNonce()` and `CmdAssociateAddress()` were made NOTE: the latter `CmdAssociateAddress` had an internal cmd method call to `sei_associate` using GET instead of POST. Fixes REL-107 (cherry picked from commit 7c036d1)
|
The latest Buf updates on your PR. Results from workflow Buf / buf (pull_request).
|
Codecov Report❌ Patch coverage is
❌ Your patch status has failed because the patch coverage (66.66%) is below the target coverage (70.00%). You can increase the patch coverage or adjust the target coverage. Additional details and impacted files@@ Coverage Diff @@
## release/v6.3 #2687 +/- ##
================================================
+ Coverage 43.32% 43.34% +0.01%
================================================
Files 1811 1866 +55
Lines 149085 155279 +6194
================================================
+ Hits 64598 67310 +2712
- Misses 78811 81959 +3148
- Partials 5676 6010 +334
Flags with carried forward coverage won't be shown. Click here to find out more.
🚀 New features to boost your workflow:
|
Backport of #2675 to
release/v6.3.