Niklas Adolfsson authored Dec 06, 2021 and Niklas Adolfsson committed Dec 07, 2021

* rpc module: add `call_and_subscribe`

* Update utils/src/server/rpc_module.rs

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

* Update types/src/v2/response.rs

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

* grumbles

* fix rustdoc links

* 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]>

* Squashed MethodSink

* Middleware WIP

* Passing all the information through

* Unnecessary `false`

* Apply suggestions from code review

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

* Add a setter for middleware (#577)

* Fix try-build tests

* Add a middleware setter and an example

* Actually add the example

* Grumbles

* Use an atomic

* Set middleware with a constructor instead

* Resolve a todo

* Update ws-server/src/server.rs

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

* Update ws-server/src/server.rs

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

* Update ws-server/src/server.rs

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

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

* Middleware::on_response for batches

* Middleware in HTTP

* fmt

* Server builder for HTTP

* Use actual time in the example

* HTTP example

* Middleware to capture method not found calls

* An example of adding multiple middlewares. (#581)

* Add an example of adding multiple middlewares.

* Update examples/multi-middleware.rs

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

* Update examples/Cargo.toml

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

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

* Move `Middleware` to jsonrpsee-types (#582)

* Move `Middleware` to jsonrpsee-types

* Move Middleware trait to jsonrpsee-types

* Add some docs.

* Link middleware to `with_middleware` methods in docs

* Doctests

* Doc comment fixed

* Clean up a TODO

* Switch back to `set_middleware`

* fmt

* Tests

* Add `on_connect` and `on_disconnect`

* Add note to future selves

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

* fix(types): use `Cow` for deserializing `str`

* use ToString

* fix rpc error: support unquoted strings

* Update types/src/v2/error.rs

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

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

* Add CODEOWNERS file

* Fix typo

* Add tools-team as main codeowner

* Bump version –> 0.5
Fix try-build tests

* Changelog

* Update CHANGELOG.md

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

* Update CHANGELOG.md

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

* Update hyper-rustls requirement from 0.22 to 0.23

Updates the requirements on [hyper-rustls](https://github.com/ctz/hyper-rustls) to permit the latest version.
- [Release notes](https://github.com/ctz/hyper-rustls/releases)
- [Commits](https://github.com/ctz/hyper-rustls/compare/v/0.22.0...v/0.23.0

)

---
updated-dependencies:
- dependency-name: hyper-rustls
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <[email protected]>

* make it work

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Niklas Adolfsson <[email protected]>

* proc-macros: Fix documentation typo of `rpc_identifier`

* proc-macros: Support deprecated methods for rpc client (#564)

Calling a deprecated method of the RPC client should warn
the user at compile-time.

Extract the `#[deprecated]` macro as is while parsing the
RpcMethod, and pass through the macro to the RPC client
rendering.

* tests/ui: Check deprecated method for rpc client (#564)

To ensure that the test will fail during compilation,
warnings are denied.

Check that the deprecate macro will generate warnings
just for the methods that are utilized.

* feat: override `method` subscription notif

* Arrow syntax for overwrites (#569)

* check that unique notifs are used

* check that custom sub name is unique

* cargo fmt

* address grumbles

* Update proc-macros/src/rpc_macro.rs

* commit added tests

* Update proc-macros/src/render_server.rs

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

* Update proc-macros/src/render_server.rs

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

* Update proc-macros/src/rpc_macro.rs

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

* Update proc-macros/src/rpc_macro.rs

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

* Update utils/src/server/rpc_module.rs

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

* grumbles

* fix long lines

* 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 proc-macros/src/rpc_macro.rs

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

* Update proc-macros/src/render_server.rs

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

* Update proc-macros/src/render_server.rs

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

* more grumbles

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

* fix(TestSubscription): use None for closed.

* add test for subscription close

* rpc module: report error on invalid subscription

* fix tests

* remove some boiler plate

* remove unused code

Closes https://github.com/paritytech/jsonrpsee/issues/448

This PR adds an implementation for `SubscriptionClient` to the `HttpClient` struct, which makes it possible for http clients to use macro-generated RPC servers. If an http client tries to set up a subscription it will fail with a `HttpNotImplemented` error.

* fix: enable several sockaddress when starting servers

* nits

* more verbose asserts in rustdoc tests

* fix tests

* fix tests again

* Fix some clippy issues

* Add an interval to periodically wake the SelectDriver Waker

* Apply suggestions from code review

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

* Naming grumbles

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

* ws server: don't kill connection max limit exceeds

* Update ws-server/src/server.rs

* actually use max size in soketto

* rewrite me

* improve logs

* use soketto fix

* rewrite me

* fix nit

* revert unintentional change

* use soketto 0.7.1

* fix logger

* Update ws-server/src/server.rs

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

* Update ws-server/src/server.rs

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

* Update ws-server/src/server.rs

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

* Update ws-server/src/server.rs

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

* Update ws-server/src/server.rs

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

* fix build

* reject too large response

* fix some DRY code

* feat: bounded serializer for RpcModule

* Update utils/src/server/helpers.rs

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

* Update utils/src/server/helpers.rs

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

* address grumbles: safety and other nits

* address grumbles: MaxCallSize on closures instead

* Update utils/src/server/helpers.rs

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

* use max response size on errors too

* Revert "use max response size on errors too"

This reverts commit 3b07e42d257b2eebae311b92b7f72594d94d5f87.

* include max limit in error response

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

* Array syntax aliases

* fmt

* ws server: don't kill connection max limit exceeds

* Update ws-server/src/server.rs

* actually use max size in soketto

* rewrite me

* improve logs

* use soketto fix

* rewrite me

* fix nit

* revert unintentional change

* use soketto 0.7.1

* fix logger

* Update ws-server/src/server.rs

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

* Update ws-server/src/server.rs

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

* Update ws-server/src/server.rs

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

* Update ws-server/src/server.rs

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

* Update ws-server/src/server.rs

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

* fix build

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

* docs(release): improve release checklist

* Update RELEASE-CHECKLIST.md

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

* Update RELEASE-CHECKLIST.md

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

* Update RELEASE-CHECKLIST.md

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

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

Bumps [actions/checkout](https://github.com/actions/checkout) from 2.3.5 to 2.4.0.
- [Release notes](https://github.com/actions/checkout/releases)
- [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md)
- [Commits](https://github.com/actions/checkout/compare/v2.3.5...v2.4.0

)

---
updated-dependencies:
- dependency-name: actions/checkout
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Implement Future for server handles

* Explicitly assert timeout errors in tests

* request ID as RAII guard

* clippify

* fmt

* address grumbles: naming

`RequestIdGuard` -> `RequestIdManager`
`RequestId` -> `RequestIdGuard`

* ci: test each feature individually

* fix nit: --all-targets is an arg

* fix rustdoc link

* get rid of cargo hack test; too slow

* feat(proc_macro): add support for map arguments

* feat(proc_macro): formatting

* feat(proc_macro): fix issues with Into trait

* feat(proc_macro): param_format for methods

* feat(proc_macro): improve param_format checking

- Addressed @niklasad1

's suggestion to use an Option instead of just
defaulting to "array".

* feat(proc_macro): apply suggestions, add test case

- Use enum for param format.
- Extract parsing logic into separate function.
- Add ui test.

* feat(proc_macro): run cargo fmt

* feat(proc_macro): address suggestions

* feat(proc_macro): document param_kind argument

* feat(proc_macro):  consistent spacing

Apply @maciejhirsz formatting suggestion.

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

* feat(proc_macro): apply suggestions

- make parameter encoding DRY
- remove strings from param_kind
- return result from parse_param_kind

* feat(proc_macro): formatting

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

* reproduce Kian's issue

* fix ws server wait until batches has completed

* fix nit

* clippify

* enable benches for ws batch requests

* use stream instead of futures::join_all

* clippify

* address grumbles: better assert

* Update tokio-rustls requirement from 0.22 to 0.23

Updates the requirements on [tokio-rustls](https://github.com/tokio-rs/tls) to permit the latest version.
- [Release notes](https://github.com/tokio-rs/tls/releases)
- [Commits](https://github.com/tokio-rs/tls/commits

)

---
updated-dependencies:
- dependency-name: tokio-rustls
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <[email protected]>

* push fixes but requires rustls-native-certs v0.6

* update native certs to 0.6.0

* fix clippy warnings

* remove webpki roots support

* Revert "remove webpki roots support"

This reverts commit 1144d567b343049ab7c967d320fc2fe162ba0f7c.

* support both native cert store and webpki

* sort deps in Cargo.toml

* Update ws-client/src/transport.rs

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

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Niklas Adolfsson <[email protected]>
Co-authored-by: David <[email protected]>

* feat(ws client): support custom headers.

Closing #531

* remove empty line

* address grumbles: more user-friendly API

* Update ws-client/src/client.rs

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

* Update ws-client/src/client.rs

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

* Update ws-client/src/client.rs

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

* address grumbles

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

Bumps [actions/checkout](https://github.com/actions/checkout) from 2.3.4 to 2.3.5.
- [Release notes](https://github.com/actions/checkout/releases)
- [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md)
- [Commits](https://github.com/actions/checkout/compare/v2.3.4...v2.3.5

)

---
updated-dependencies:
- dependency-name: actions/checkout
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* ci: change directory structure

* fix build

* fix http client bench with request limit

* benches for jsonrpc servers

* workaround; dont use max request limit

* add subscriptions

* revert unintentional change

* ignore batch request bench for ws

* fmt

* log -> tracing

* test bench CI

* test bench v0.3

* wtf; run CI

* work plz

* remove test CI bench

* fix compile warn on macos

* Re-introduce object param parsing

* fmt and unnecessary collect