-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
rpc/jsonrpc/server: return an error in WriteRPCResponseHTTP(Error) #6204
Conversation
instead of panicking Closes #5529
Codecov Report
@@ Coverage Diff @@
## master #6204 +/- ##
==========================================
+ Coverage 60.72% 60.79% +0.06%
==========================================
Files 276 276
Lines 25788 25788
==========================================
+ Hits 15659 15677 +18
+ Misses 8508 8487 -21
- Partials 1621 1624 +3
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🔥
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🎉
Co-authored-by: Aleksandr Bezobchuk <alexanderbez@users.noreply.github.com>
Am I correct in thinking this will fix my validator going MIA for a brief period (a block or two) with repeating
Never mind: I saw that #6078 and #5529 together described my situation, and the current PR looks like it solves it completely. |
…#6204) (#6230) * rpc/jsonrpc/server: return an error in WriteRPCResponseHTTP(Error) (#6204) instead of panicking Closes #5529 (cherry picked from commit 00b9524) # Conflicts: # CHANGELOG_PENDING.md # rpc/jsonrpc/server/http_json_handler.go # rpc/jsonrpc/server/http_server.go # rpc/jsonrpc/server/http_server_test.go # rpc/jsonrpc/server/http_uri_handler.go * resolve conflicts * fix linting * fix conflict Co-authored-by: Anton Kaliaev <anton.kalyaev@gmail.com> Co-authored-by: Marko Baricevic <marbar3778@yahoo.com>
instead of panicking
Closes #5529