* [ws server]: refactor start()

* remove needless clone

* fix http server too

* fmt

* unify stop APIs

* fix bad merge

* add API to use custom tokio runtime

* use futures oneshot

* fix unintentional change

* Revert "use futures oneshot"

This reverts commit 0432f6ba41d49a3494943458657a31f8afd79160.

* Update http-server/src/server.rs

Co-authored-by: Maciej Hirsz <[email protected]>

Co-authored-by: Maciej Hirsz <[email protected]>

* use similar API for HTTP ACL builder

* revert unintentional change

* fix nits

* Update http-server/src/access_control/mod.rs

Co-authored-by: David <[email protected]>

* grumbles

Co-authored-by: David <[email protected]>

* Squash post merge

* RAII claims and some comments

* Adding resources to Methods

* Use arrays for claiming all resources at once

* Register resources on ws-server

* Return MethodResourcesBuilder from register_method

* Return MethodResourcesBuilder from register_async_method

* fmt and comments

* Add `verify_and_insert` to get rid of `or_insert` in order to get mut ref to value

* Resources initialization

* No need for ResourceBuilder

* Comment all the things

* Handle errors on ws-server `start` method

* Passing ResourceGuard to method execution

* Start ws-server in background, simplify all the tests!

* Claiming resources on runtime

* Do checked math on resource claiming

* Clearer error path for resources at limit

* Resource limiting tests WIP

* Unbox async futures

* fmt

* Cleaner proc macro

* Fix lifetime issues with ResourceGuard with internal Arc

* Remove unnecessary lifetime generic

* Working tests

* Use turbofish in tests

* fmt

* Fixed (?) benches

* Apply suggestions from code review

Co-authored-by: David <[email protected]>

* Tweak more grumbles

* Adding a test for resource limiting with proc macros (WIP)

* Reworking attribute macros for resource limiting (#507)

* Squashed macros WIP

* bae-less macros!

* Make things simpler and more readable

* Some comments and DRY aliases parsing

* Naming things is hard

* Respan is no longer needed

* Simpler Arguments

* Remove stale code

* Apply suggestions from code review

Co-authored-by: Niklas Adolfsson <[email protected]>

* syn-up all the things, handle resources on methods

Co-authored-by: Niklas Adolfsson <[email protected]>

* Resource limiting on the http server

* Test two resources at once

* Test the HTTP server for resource limiting

* Apply suggestions from code review

Co-authored-by: David <[email protected]>
Co-authored-by: Niklas Adolfsson <[email protected]>

* Comment tweaks

* More explicit error type for `AttributeMeta::retain`

* Remove generic from ResourceTable; derive Default on ws Builder

* Make the macros shorter

Co-authored-by: David <[email protected]>
Co-authored-by: Niklas Adolfsson <[email protected]>

* feat(ws client): support redirections

* reuse socket

* reuse socket

* add hacks

* fix build

* remove hacks

* fix bad merge

* address grumbles

* fix grumbles

* fix grumbles

* fix nit

* add redirection test

* Update test-utils/src/types.rs

* Resolved todo

* Check that redirected client actually works

* Rename test-utils "types" to "mocks"

* Fix windows test (?)

* fmt

* What is wrong with you windows?

* Ignore redirect test on windows

* fix bad transport errors

* debug windows tests

* update soketto

* maybe fix windows test

* add config flag for max redirections

* revert faulty change.

Relative reference must start with either `/` or `//`

* revert windows path

* use manual join paths

* remove url dep

* Update ws-client/src/tests.rs

* default max redirects 5

* remove needless clone vec

* fix bad merge

* cmon CI run

Co-authored-by: David Palm <[email protected]>

* Unbox async futures

* fmt

* Cleaner proc macro

* Remove unnecessary parens around async blocks

* Re-export v2::params, error, request and response types

* Use shallow type paths

* Renames

* More renames

* fmt

* s/RpcParams/Params/

* Add a type alias `SubscriptionResponse` to `Notification` which is parametrized to `SubscriptionPayload`
Move `SubscriptionResponse` to the `response` module

* Fix doc links

* Cleanup and some docs

* Review feedback

* cleanup

* [http server]: export acl + kill cors_max_age

* fix cors_max_age nit

* fmt

* remove max_cors_age

* Silence warning on cors_max_age

* Revert change to `AccessControl`

* Add a test illustrating how to use the `call` convenience method

* Extend test + review feedback

* log

* log

* log

* log

* log

* log

* log

* log

* log

* log

* log

* log

* log

* log

* log

* Add Methods::test_subscription
Log more when params parsing fails

* Add call_with test helper (ty @niklas!) + cleanup

* Remove todo (part of https://github.com/paritytech/jsonrpsee/issues/457

)

* fmt

* Let `test_subscription` be called from other crates

* Manually fix indentation

* fmt

* SSself-review grumbles

* CallError::InvalidParams carries an anyhow::Error

* fmt

* Tweak docs

* Update utils/src/server/rpc_module.rs

Co-authored-by: Niklas Adolfsson <[email protected]>

* review grumble

* Fix todos

* fmt

* Fixup error messages

* Include source in the error message for `CallError`

* fmt

* Update proc-macros/src/render_server.rs

Co-authored-by: Maciej Hirsz <[email protected]>

* Mention needing jsonrpsee crate in scope

* Resolve todo

* Impl ToRpcParams for 0-sized array

* optimized logging

Co-authored-by: Niklas Adolfsson <[email protected]>
Co-authored-by: Maciej Hirsz <[email protected]>

* cleanup

* Update ws-server/src/tests.rs

* Update http-server/src/tests.rs

* Add license headers where missing and update year

* Update examples/http.rs

Co-authored-by: Niklas Adolfsson <[email protected]>

Co-authored-by: Niklas Adolfsson <[email protected]>

* Experimental generic driver for methods

* Move FutureDriver to its own module

* "Infallible" sync methods + fmt

* Drive all methods to completion before background_task closes

* Sniff the first byte to glean if the incoming request is a single or batch request

This works around the serde limitations around `untagged` enums and `RawValue`.

* fmt

* For http server, check first byte before allocating space for the body

Also, rework the way we return errors: prefer JSON-RPC errors according to spec (application/json) wherever sensible.

* Review feedback

* Don't assume there is a first byte to read

* ty clipyp

* Review concerns

* Cleanup

Co-authored-by: David Palm <[email protected]>

* Sniff the first byte to glean if the incoming request is a single or batch request

This works around the serde limitations around `untagged` enums and `RawValue`.

* fmt

* Review feedback

* Don't assume there is a first byte to read

* remove todo

* review feedback

* Do not register methods on servers

* fmt

* Infallible `to_rpc` proc macro

* Remove dead code

* Check for duplicate names at compile time

* Add a UI test for name conflicts

* Apply suggestions from code review

Co-authored-by: David <[email protected]>

Co-authored-by: David Palm <[email protected]>

* Include "macros" in the "server" feature

* Re-export SubscriptionSink

* Include the "types" feature in both the "client" and "server" features
Export types::* from façade when the "types" is active
Export types::* from servers

* fmt

* Export jsonrpsee_types under types

* fmt

* broken doc link

* fix benches

* fmt

* Fix test imports

* Add a way to stop HTTP server

* Add a way to stop WS server

* Apply suggestions from code review

Co-authored-by: David <[email protected]>

* Ensure the concrete type of error in stop test

* Resolve merge artifacts

* Add public re-exports of stop handle

Co-authored-by: David <[email protected]>

* [jsonrpsee types]: unify types + more tests

* address grumbles

* Add a builder to the ws server

* Use the WsServerBuilder in tests&examples

* cleanup

* fmt

* Fix benches

* fix error message

* Add connection count limiting and associated config item

* Use the strong_count to `methods` to know how many connections we currently have

* Fix test

* Use a single map for methods

* fmt

* cargo fmt screwed indents somehow

* Use Arc instead of Box for async methods

* fmt

* Implement `execute` on `MethodCallback` to keep things DRY

* fmt

* `OwnedJsonRpcRequest` is not necessary anymore

* Call `execute` on `Methods` directly for even more DRY code

* Update utils/src/server/rpc_module.rs

Co-authored-by: Niklas Adolfsson <[email protected]>

* Use `copied` instead of mapping a deref

* DRY-er `params` in `MethodCallback`

* Added comment on `0` conn_id in http

Co-authored-by: Niklas Adolfsson <[email protected]>

* Enable docs/debug/pub warnings for server and types crates

* Remove Debug bound on Notif

* Stick to deriving debug

* Update utils/src/server/rpc_module.rs

Co-authored-by: David <[email protected]>

Co-authored-by: David <[email protected]>

* Misc async methods tinkerings

* Update utils/src/server/rpc_module.rs

Co-authored-by: Niklas Adolfsson <[email protected]>

* Update types/src/v2/params.rs

Co-authored-by: Niklas Adolfsson <[email protected]>

* Fix review grumbles

* fmt

Co-authored-by: Niklas Adolfsson <[email protected]>

* Add basic support for async methods

* Add support of async methods into ws server

* Add support of async methods to HTTP server

* Fix subscriptions method lookup

* Remove type that appeared to be unused

* Add comment on safety of unwrap in OwnedJsonRpcRequest::borrowed

* Update utils/src/server/rpc_module.rs

Co-authored-by: Niklas Adolfsson <[email protected]>

Co-authored-by: Maciej Hirsz <[email protected]>
Co-authored-by: Niklas Adolfsson <[email protected]>

* Remove register_* from Server

* Tests pass

* Some cleanup

* more cleanup

* Less bounds

* Remove the `RpcMethod` trait – I don't think we need it.

* fmt

* Fix benches

* Enable merging two modules with different contexts

* Module registration returns number of methods on the server and checks for dupes
Tests for module registration and some cleanup

* Export `RpcModule` from façade crate
fmt

* Revert to return error from `Server::register_module`

* Remove Server::methods

* Remove TODO

* Fix tests
Fix http server to work the same as ws-server

* Server::method_names

* fmt

* fix clippy warnings

Co-authored-by: Niklas Adolfsson <[email protected]>

* Switch from anyhow::Result to std Result

* fmt

* Use thiserror `#[from]`

* Remove sadness

* Add extension trait to timeout the futures execution

* Use timeout for ws-server tests

* Use timeout for ws-client tests

* Use timeout for http-server tests

* Use timeout for http-client tests

* Make TimeoutFutureExt interafaces more explicit

* get started

* add additional test

* fix nits

* cargo fmt

* [types]: write some tests.

* [http server]: send empty response on notifs

* [http server]: fix tests

* [rpc module]: send subscription response

* Update types/src/v2/error.rs

* fix nits

* cargo fmt

* Update types/src/v2/params.rs

* remove needless clone

* remove dead code

* [types]: impl PartialEq for JsonErrorObject + test

* use beef::Cow

* Update http-server/src/tests.rs

* [rpc server]: extract rpc_module to utils.

This commit extracts the `rpc_module` from the servers to be shared.
It will help to re-use rpc modules within both the servers

* Update utils/src/server/rpc_module.rs

Co-authored-by: David <[email protected]>

* Update utils/src/server/rpc_module.rs

Co-authored-by: David <[email protected]>

* Update utils/src/server/mod.rs

Co-authored-by: David <[email protected]>

* Update utils/src/server/rpc_module.rs

Co-authored-by: David <[email protected]>

* Update utils/src/server/rpc_module.rs

Co-authored-by: David <[email protected]>

* Update utils/src/server/rpc_module.rs

Co-authored-by: David <[email protected]>

* Update utils/src/server/rpc_module.rs

Co-authored-by: David <[email protected]>

* Update utils/src/server/rpc_module.rs

Co-authored-by: David <[email protected]>

* Update utils/src/server/rpc_module.rs

Co-authored-by: David <[email protected]>

* Update utils/src/server/rpc_module.rs

Co-authored-by: David <[email protected]>

* address grumbles

* fix build

* fix docs

* cargo fmt

* Update utils/src/server/rpc_module.rs

Co-authored-by: David <[email protected]>

* grumbles: use MethodSink

* Update utils/src/server/mod.rs

Co-authored-by: David <[email protected]>

* Update utils/src/server/rpc_module.rs

Co-authored-by: David <[email protected]>

* Update utils/src/server/rpc_module.rs

Co-authored-by: David <[email protected]>

Co-authored-by: David <[email protected]>

* Add a test for calling methods with multiple params of multiple types

* cargo fmt

Co-authored-by: Niklas Adolfsson <[email protected]>

* Batch requests over the websocket

* Use same call signature for both http and ws server

* Use CallError instead of InvalidParams
Add a draft batch request test

* Sort out formatting of batch responses
Cleanup
Use CallError

* Add test for slow method calls

* Update utils/src/server.rs

Co-authored-by: Niklas Adolfsson <[email protected]>

* Add benchmark for batched websocket requests

* rename

* Rename batch channels

* Sort out the InvalidParams situation

* Remove one level of InvalidParams
Return app-level error when call fails

* Update ws-server/src/server.rs

Co-authored-by: Andrew Plaza <[email protected]>

* Update ws-server/src/server.rs

Co-authored-by: Andrew Plaza <[email protected]>

Co-authored-by: Niklas Adolfsson <[email protected]>
Co-authored-by: Andrew Plaza <[email protected]>

* ret err if context/params fails

* address grumbles: specific error_code context fail

* address grumbles: make env_logger dev-dependency

* address grumbles: add tests

* chore(deps): remove unused deps

* address grumbles: rename types and docs

* address grumbles: more renaming.

* fix build

* WIP

* Implement draft of batch requests

* fmt

* cleanup

* Explain why we don't use an untagged enum

* Avoid allocating a Vec for single requets

* Add comment

* Add a benchmark for batch requests

* Add more tests, noting where we diverge from the spec
Fix empty batch case, i.e. `[]`

* Obey the fmt

* Update benches/bench.rs

Co-authored-by: Andrew Plaza <[email protected]>

* Update http-server/src/server.rs

Co-authored-by: Andrew Plaza <[email protected]>

* Add link to issue

* Explain why we're closing the receiving end of the channel.

* Limit logging of requests and response to 1kb
Add more comments
Factor out batch response collection

* Wrap comment

* tweak log line

* Benchmark batch request over different batch sizes

* fmt

Co-authored-by: Andrew Plaza <[email protected]>

* [types]: allow `data` field in jsonrpc error obj.

* fmt

* revert change to make fail

* remove redundant test

* [types]: bring back `data` and `message` in error.

Basically deducing the error message from server defined error doesn't work
and we should not strip out this information from to user which might be useful.

* remove boiler plate code

* [types]: impl From<ErrorCode> for JsonRpcError

* address grumbles

* remove more boiler plate

* Use a constant to express max request body size
Consistently use u32 in all crates

* Review grumbles