Closed zivkovicmilos closed 1 week ago
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 54.61%. Comparing base (
6032b48
) to head (4b561bf
).
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Description
This PR fixes a bug with the
http.Client
, where the customDialContext
would fail if the specified RPC URL did not contain a port number (ex:https://rpc.gno.land
).I've added the default catch values, and removed useless error returns.
This additionally begs the question -- do we even want a custom dial context, given that we don't even use the features it provides?
Contributors' checklist...
- [x] Added new tests, or not needed, or not feasible - [x] Provided an example (e.g. screenshot) to aid review or the PR is self-explanatory - [x] Updated the official documentation or not needed - [x] No breaking changes were made, or a `BREAKING CHANGE: xxx` message was included in the description - [ ] Added references to related issues and PRs - [ ] Provided any useful hints for running manual tests - [ ] Added new benchmarks to [generated graphs](https://gnoland.github.io/benchmarks), if any. More info [here](https://github.com/gnolang/gno/blob/master/.benchmarks/README.md).