Apps: Block Explorer Detailed View Failed Subscription

[yjkimjunior]

On [next] branch, when clicking through to block from explorer, Failed subscription: {code: -32601, message: "Method not found"}

Related: #278

[jacogr]

Mmmmm... this one is weird.

Are you running latest substrate master? Will double-check this one again, I know there are some errors floating around which should not be there, however for this type of thing it seems like we are using a method the Node does not have, only thing I can think of right at this point is slightly outdated Rust binaries.

[jacogr]

Ok, think you probably ran into my favourite "hiding in plain sight" bug. For BBQ, the click through from hash doesn't work, only click through from parentHash does. And I still don't have a workable work-around for this one. (For Substrate master, both work)

Basically, with the current Substrate master (and soon PoC-3), the block header is specified as blockNumber: Compact<BlockNumber> and in BBQ it is still the old blockNumber: BlockNumber - long story short, the hash gets calculated incorrectly from the serialised block. (It needs to be calculated, i.e. it is not in the response)

If this is the issue, nothing much that can be done atm - I dunno how to detect this case to hide the link or calculate correctly atm. (It has been out of my mind some days, maybe we can think of _something_ now that it is front-of-mind again)

EDIT: Still don't understand the failed method though, will have to take a peek on both versions and see if I can see the same

[jacogr]

Ok, checked against both BBQ Birch (hosted RPC) and local node (recent substrate master), cannot see the same RPC "method not found" error. (The above block-click-through bug is there though)

[yjkimjunior]

Locally I am on the latest Substrate master but realized it's not from clicking through in the explorer. It's just a warning that comes up when any page loads. Same with the hosted BBQ Birch RPC. Seems it's something else from the bonds library? Doesn't seem as important as the issue with blockNumber: Compact<BlockNumber> though.

Full stack trace:

```
subscriptionBond.js?3a42:27 Failed subscription: {code: -32601, message: "Method not found"}code: -32601message: "Method not found"__proto__: Object

subscription.nodeService.subscribe.error | @ | subscriptionBond.js?3a42:27
-- | -- | --
  | Promise.catch (async) |   |  
  | subscribe | @ | nodeService.js?6adf:148
  | initialise | @ | subscriptionBond.js?3a42:25
  | use | @ | bond.js?95ac:347
  | tie | @ | bond.js?95ac:478
  | (anonymous) | @ | bonds.js?6561:39
  | ../../node_modules/oo7-substrate/src/bonds.js | @ | main.43115bc2.js:5603
  | __webpack_require__ | @ | runtime.78548093.js:783
  | fn | @ | runtime.78548093.js:151
  | (anonymous) | @ | transact.js?af99:6
  | ../../node_modules/oo7-substrate/src/transact.js | @ | main.43115bc2.js:5809
  | __webpack_require__ | @ | runtime.78548093.js:783
  | fn | @ | runtime.78548093.js:151
  | (anonymous) | @ | index.js?0417:6
  | ../../node_modules/oo7-substrate/src/index.js | @ | main.43115bc2.js:5636
  | __webpack_require__ | @ | runtime.78548093.js:783
  | fn | @ | runtime.78548093.js:151
  | (anonymous) | @ | index.js?00e3:3
  | ../../node_modules/polkadot-identicon/lib/index.js | @ | main.43115bc2.js:6164
  | __webpack_require__ | @ | runtime.78548093.js:783
  | fn | @ | runtime.78548093.js:151
  | (anonymous) | @ | Polkadot.tsx?1fd6:12
  | ../ui-react/src/IdentityIcon/Polkadot.tsx | @ | main.43115bc2.js:10012
  | __webpack_require__ | @ | runtime.78548093.js:783
  | fn | @ | runtime.78548093.js:151
  | (anonymous) | @ | index.tsx?7f6c:18
  | ../ui-react/src/IdentityIcon/index.tsx | @ | main.43115bc2.js:10036
  | __webpack_require__ | @ | runtime.78548093.js:783
  | fn | @ | runtime.78548093.js:151
  | (anonymous) | @ | KeyPair.tsx?3b05:16
  | ../ui-keyring/src/options/KeyPair.tsx | @ | main.43115bc2.js:9605
  | __webpack_require__ | @ | runtime.78548093.js:783
  | fn | @ | runtime.78548093.js:151
  | (anonymous) | @ | item.tsx?4afa:14
  | ../ui-keyring/src/options/item.tsx | @ | main.43115bc2.js:9629
  | __webpack_require__ | @ | runtime.78548093.js:783
  | fn | @ | runtime.78548093.js:151
  | (anonymous) | @ | genericSubject.ts?c147:18
  | ../ui-keyring/src/observable/genericSubject.ts | @ | main.43115bc2.js:9570
  | __webpack_require__ | @ | runtime.78548093.js:783
  | fn | @ | runtime.78548093.js:151
  | (anonymous) | @ | accounts.ts?e4fa:12
  | ../ui-keyring/src/observable/accounts.ts | @ | main.43115bc2.js:9534
  | __webpack_require__ | @ | runtime.78548093.js:783
  | fn | @ | runtime.78548093.js:151
  | (anonymous) | @ | index.ts?7027:26
  | ../ui-keyring/src/index.ts | @ | main.43115bc2.js:9522
  | __webpack_require__ | @ | runtime.78548093.js:783
  | fn | @ | runtime.78548093.js:151
  | (anonymous) | @ | index.tsx?2f5d:28
  | ../ui-react-rx/src/Api/index.tsx | @ | main.43115bc2.js:9653
  | __webpack_require__ | @ | runtime.78548093.js:783
  | fn | @ | runtime.78548093.js:151
  | (anonymous) | @ | index.ts?210d:75
  | ../ui-react-rx/src/index.ts | @ | main.43115bc2.js:9785
  | __webpack_require__ | @ | runtime.78548093.js:783
  | fn | @ | runtime.78548093.js:151
  | (anonymous) | @ | index.tsx?df60:196
  | ../ui-app/src/index.tsx | @ | main.43115bc2.js:9229
  | __webpack_require__ | @ | runtime.78548093.js:783
  | fn | @ | runtime.78548093.js:151
  | (anonymous) | @ | index.tsx?7604:9
  | ./src/index.tsx | @ | main.43115bc2.js:10282
  | __webpack_require__ | @ | runtime.78548093.js:783
  | fn | @ | runtime.78548093.js:151
  | 1 | @ | main.43115bc2.js:10484
  | __webpack_require__ | @ | runtime.78548093.js:783
  | checkDeferredModules | @ | runtime.78548093.js:46
  | webpackJsonpCallback | @ | runtime.78548093.js:33
  | (anonymous) | @ | main.43115bc2.js:1

[jacogr]

Ok, so why are bonds popping up here? We are not actually using them - must be a dep from the icon.

I have not seen that, but what I have been seeing is duplicate subscriptions to localhost when I'm connecting to BBQ.

Either way, this is not quite cool :(

[jacogr]

@yjkimjunior What UI theme are you using - the Polkadot or Substrate one?

[yjkimjunior]

@jacogr I'm using the Polkadot theme, just checked on Safari and FF as well just in case - same story.

[yjkimjunior]

So we're using bonds as a side effect of including polkadot-identicon lib. Anyway from digging in a bit it looks like the error pops up from trying to subscribe to chain_runtimeVersion which should now be chain_getRuntimeVersion.

[jacogr]

Removed deps in the above PR, which meant including the actual rendering code without any Reactive deps. (Some positive side effects as well, the dependency tree has cleaned up)

[polkadot-js-bot]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue if you think you have a related problem or query.