tendermint

Commit Graph

Author	SHA1	Message	Date
Erik Grinaker	531fa07ad2	rpc: fix test data races (#5363 ) Fixes a couple of test data races. `testing.T` is not concurrency-safe and should not be used in goroutines at all.	4 years ago
Marko	0ed8dba991	lint: enable errcheck (#5336 ) ## Description Enable errcheck linter throughout the codebase Closes: #5059	4 years ago
Marko	2d167aefcf	ci: freeze golangci action version (#5196 ) ## Description This PR updates golang-ci to latest and stops looking at master for the action. Closes: #XXX	4 years ago
Anton Kaliaev	95fc7e58ee	rpc/jsonrpc/server: merge WriteRPCResponseHTTP and WriteRPCResponseAr (#5141 ) ...rrayHTTP Closes #5135 Also, wrote a test for WriteRPCResponseHTTPError and used it with correct status codes according to https://www.jsonrpc.org/historical/json-rpc-over-http.html#response-codes	4 years ago
Marko	7e2cc1db5e	linter: (1/2) enable errcheck (#5064 ) ## Description partially cleanup in preparation for errcheck i ignored a bunch of defer errors in tests but with the update to go 1.14 we can use `t.Cleanup(func() { if err := <>; err != nil {..}}` to cover those errors, I will do this in pr number two of enabling errcheck. ref #5059	4 years ago
Anton Kaliaev	a14ff5cb30	rpc: refactor lib folder (#4836 ) Closes https://github.com/tendermint/tendermint/issues/3857 Moves `lib/` folder to `jsonrpc/`. Renames: packages `rpc` package -> `jsonrpc` package `rpcclient` package -> `client` package `rpcserver` package -> `server` package structs and interfaces ``` JSONRPCClient to Client JSONRPCRequestBatch to RequestBatch JSONRPCCaller to Caller ``` functions ``` StartHTTPServer to Serve StartHTTPAndTLSServer to ServeTLS rpc/jsonrpc/client: rename NewURIClient to NewURI NewJSONRPCClient to New NewJSONRPCClientWithHTTPClient to NewWithHTTPClient NewWSClient to NewWS ``` misc - unexpose `ResponseWriterWrapper` - remove unused http_params.go	5 years ago
Anton Kaliaev	1b77bf6f20	rpc/lib: write a test for TLS server (#3703 ) * rpc/lib: write a test for TLS server Refs #3700 * do not regenerate certificates * add nolint	5 years ago
Ismail Khoffi	1e3469789d	Ensure WriteTimeout > TimeoutBroadcastTxCommit (#3443 ) * Make sure config.TimeoutBroadcastTxCommit < rpcserver.WriteTimeout() * remove redundant comment * libs/rpc/http_server: move Read/WriteTimeout into Config * increase defaults for read/write timeouts Based on this article https://www.digitalocean.com/community/tutorials/how-to-optimize-nginx-configuration * WriteTimeout should be larger than TimeoutBroadcastTxCommit * set a deadline for subscribing to txs * extract duration into const * add two changelog entries * Update CHANGELOG_PENDING.md Co-Authored-By: melekes <anton.kalyaev@gmail.com> * Update CHANGELOG_PENDING.md Co-Authored-By: melekes <anton.kalyaev@gmail.com> * 12 -> 10 * changelog * changelog	6 years ago
Alessio Treglia	b646437ec7	Decouple StartHTTP{,AndTLS}Server from Listen() (#2791 ) * Decouple StartHTTP{,AndTLS}Server from Listen() This should help solve cosmos/cosmos-sdk#2715 * Fix small mistake * Update StartGRPCServer * s/rpc/rpcserver/ * Start grpccore.StartGRPCServer in a goroutine * Reinstate l.Close() * Fix rpc/lib/test/main.go * Update code comment * update changelog and comments * fix tm-monitor. more comments	6 years ago
Alessio Treglia	f11db8c1b0	Pass http.ServeTLS() errors back to the caller (#2461 ) Closes: #2460 * Pass http.ServeTLS() errors back to the caller * Update CHANGELOG * Amend StartHTTPServer() too for consistency's sake * Revert "Amend StartHTTPServer() too for consistency's sake" This reverts commit `23bfb4c2e9`.	6 years ago
Zach Ramsay	44dad6d70b	Revert "detele everything" This reverts commit `d02c5d1e30`.	6 years ago
Zach Ramsay	d02c5d1e30	detele everything	6 years ago
Ethan Buchman	d55243f0e6	fix import paths	6 years ago
Anton Kaliaev	936a655990	limit number of open connections Refs #1740 also, expose limit option for number concurrent streams for gRPC (unlimited by default)	7 years ago

6 Commits (093dcfc8a0c5cb975b89119d5c797a1bcabeaea4)