search

paritytech / polkadot-sdk

The Parity Polkadot Blockchain SDK
https://polkadot.network/
1.81k stars 659 forks source link

Consider restricting cumulus access to unnecessary runtime APIs #1332

Open alindima opened 1 year ago

alindima commented 1 year ago

Context: https://github.com/paritytech/polkadot-sdk/pull/1200#discussion_r1311327094

This would mean adding a more stripped down variant of the RuntimeApiSubsystemClient trait.

bkchr commented 1 year ago

@sandreim: I don't fully agree with that. For UX simplicity, I think the interface should expose the bare necessities for collators.

Valid argument, but then every subsystem needs to expose what kind of runtime apis it is calling without having one central runtime api instance.