Skip to content
  • Hernando Castano's avatar
    Use Subscription Manager from `jsonrpc-pubsub` (#6208) · f028a509
    Hernando Castano authored
    * Bump jsonrpc pubsub, core, http, and ws
    
    Right now these are the packages which _need_ to be updated
    so I can just the latest `jsonrpc-pubsub` code. Once a release
    it cut upstream the rest of the dependencies should be updated
    as well.
    
    * Use jsonrpc-pubsub's SubscriptionManager
    
    This places sc-rpc-api::Subscriptions
    
    * Bump jsonrpc-core outside of sc-rpc-*
    
    * Update client/rpc tests
    
    Right now one of the `author` tests is failing, I
    need to think a bit about how best to fix it.
    
    * Remove Subscriptions manager
    
    There's no need for this implementation since we're
    using the one from `jsonrpc-pubsub` now
    
    * Fix author RPC test
    
    This test used to check for a numerial subscription ID,
    whereas now it uses a string based ID which is the default
    provided by `jsonrpc-pubsub`'s subscription manager.
    
    * Remove unused NumericIdProvider
    
    * Add missing bracket
    
    Removed one too many with that last one, lol
    
    * Bump `jsonrpc` to v14.2
    
    There's an exception though. `jsonrpc-derive` cannot be bumped
    past v14.0.5 just yet since it has a dependency on `quote` pinned
    to v1.0.1. This means that at the moment it won't build on Substrate
    since it's using v1.0.3.
    
    * Track `jsonrpc-derive` master branch
    
    * Bump `quote` version to v1.0.6
    
    * Bump `jsonrpc-derive` to v14.2.1
    
    This includes support for `quote` v1.0.6
    f028a509