carp icon indicating copy to clipboard operation
carp copied to clipboard

Easier switch for BearerKind

Open SebastienGllmt opened this issue 3 years ago • 2 comments

Currently the codebase assumes you're connecting to a local node. If you're connecting to a remote node (TCP connection,) you can change the BearerKind to TCP in the code.

It's not great to force people to change code for what should be a config change.

SebastienGllmt avatar May 06 '22 07:05 SebastienGllmt

Probably we can specify this in the execution plan so that we can use this as a single source of truth instead of adding other configs. .ini files support a "general section" for this kind of config

SebastienGllmt avatar May 09 '22 16:05 SebastienGllmt

One way we could solve this is by switching the dependency order. Instead of Carp taking in am execution plan as an argument, it would instead take in a config file that specifies things like BearerKind, etc. and also specify which EP to use for this config.

The good thing abour this solution is that it makes it easier to do https://github.com/dcSpark/carp/issues/41

Badically everybody can share their EPs freely and then in your config file you can specify the path or url to fetch the EP from

SebastienGllmt avatar May 29 '22 22:05 SebastienGllmt