* Document resource limiting

* Formatting

* More formatting

* fmt

* Doc comment wording

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

* Cross-reference docs between servers and utils

* Fix grumbles

* More elaborate description

* Update http-server/Cargo.toml

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

* Apply suggestions from code review

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

* Allow for unlimited resources if cap is 0

* fmt

* Update ws-server/Cargo.toml

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

* Clarify 8 resources, 0s in costs/limits, and runtime errors

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

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

* Remove unused import

* Add a simple test for angle braces in arguments

* Argument parsing should permit commas inside angle brackets

* Add a comment, move test to trybuild

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

* fix(ParamsSer): serialize to empty array

Serialize `ParamsSer::NoParams` to an empty array inorder to comply with the jsonrpc v2 spec.

* fix tests

* remove ParamsSer::NoParams

* address grumbles: adjust rpc params macro

Closing #485

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

It turns out that `hyper-rustls` already re-export ClientConfig so no need to use rustls for that purpose then.

* Update soketto requirement from 0.6 to 0.7

Updates the requirements on [soketto](https://github.com/paritytech/soketto) to permit the latest version.
- [Release notes](https://github.com/paritytech/soketto/releases)
- [Changelog](https://github.com/paritytech/soketto/blob/develop/CHANGELOG.md)
- [Commits](https://github.com/paritytech/soketto/compare/v0.6.0...v0.7.0

)

---
updated-dependencies:
- dependency-name: soketto
  dependency-type: direct:production
...

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

* set_origin -> set_headers

* fmg

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

* Add a convenience macro to build `ParamsSer` for use in clients

* Add actual macro

* fmtwtf

* Unbox async futures

* fmt

* Cleaner proc macro

* Remove unnecessary parens around async blocks

* Move RequestIdGuard to types::client

* Use the RequestIdGuard type for the http client as well

* Fix batch request ID handling (one batch uses one slot)

* Add a few tests to check that max_concurrent_requests work for http clients

* Docs

* fix: server should not send to closed subscription

Closing #481

* add a test

* rewrite me

* remove faulty server close

* fix tests

* fix tests

* remove needless code

* Update ws-client/src/client.rs

* fix: remove atomic flag; use `close_sender`

* remove terminate message; use close channel

* remove faulty subscription closed error msg

* fix tests

* fix grumbles

* fix build

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

* Reject overflowing connection with status code 429

* fmt

* rename Handshake -> HandshakeMode for clarity; verbose test

* Gracefully shutdown after rejecting to hopefully fix the errors on windows

* HandshakeMode -> HandshakeResponse; tweak pending subscriptions on shutdown test

In #461 I missed that these options are not stable so it breaks our CI. This PR reverts that change while keeping the whitespace changes from #461.

Apologies for the noise.

* new helper for testing

* fix features

* call_with_params -> call_with

* call_with_params -> call_with

* Update utils/src/server/rpc_module.rs

* fix trait bound

* Update utils/src/server/rpc_module.rs

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

* call_with -> test_call

* fix grumbles

* fix nits

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

Similar to #435 that adds the same restrictions to subscriptions too.
To avoid having faulty trait bounds on when the subcription actually returns Result.

* cleanup

* Update ws-server/src/tests.rs

* Update http-server/src/tests.rs

* fix(proc macros): generate documentation

Fixes #449

* revert proc macro example

* add test that docs are generated

* use import

* address grumbles

* make the code less ugly

* simplify more

* fix grumbles: check ident before parse

* more grumbles: filter_map -> filter

* Experimental generic driver for methods

* Move FutureDriver to its own module

* "Infallible" sync methods + fmt

* Drive all methods to completion before background_task closes

* Testing a different stop handle

* Added the waker for the monitors

* Fix the test

* Fix a possible deadlock

* fmt

* Explain strong_count check in StopHandle Drop impl

* Clippy tweak

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

* Added comment on atomicbool ordering

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

* feat: alias attribute for proc macros

* implement David's suggested no more unsub field

* keep aliases outside namespace

* revert example

* fix build

* Update proc-macros/src/attributes.rs

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

* grumbles: alias -> aliases

* grumbles v2: alias -> aliases

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

* treat empty json as no params when optional

* add test with nested array brackets

* address grumbles: better tests

* Update types/src/v2/params.rs

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

* introduce RpcParams::sequence_ignore_empty

* address grumbles: ignore empty json in RpcParams

* Update proc-macros/src/render_server.rs

* address grumbles: remove trim in next_inner()

* address grumbles: trim_start after params.next()

* address more grumbles

* [proc macros]: add test for empty array

* make proc macro kind of work

* add hack to make it work in proc macros

* add hack to make it work in proc macros

* [] ->

* revert cerebral palsy

* Update proc-macros/tests/ui/correct/basic.rs

* Update proc-macros/tests/ui/correct/basic.rs

* address grumbles

* improve is_option

* add test for core::option::Option

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