hyperledger / cacti

Hyperledger Cacti is a new approach to the blockchain interoperability problem
https://wiki.hyperledger.org/display/cactus
Apache License 2.0
339 stars 277 forks source link

[EPIC] connection chain code consolidation #1497

Open petermetz opened 2 years ago

petermetz commented 2 years ago

Description

As a maintainer I want to have packages that do not duplicate existing functionality so that it's easier to maintain the code, needs less documentation and newcomers can better understand what's going on in terms of individual package responsibilities.

This is a reboot of https://github.com/hyperledger/cactus/pull/719 which was opened in the Spring of 2021 and then went nowhere for 5 months with finally all the original work getting deleted from it completely.

Most of the issues within this epic will apply equally to all of these packages:

- packages:
  - cactus-cmd-socketio-server
  - cactus-plugin-ledger-connector-go-ethereum-socketio
  - cactus-plugin-ledger-connector-fabric-socketio
  - cactus-plugin-ledger-connector-sawtooth-socketio
izuru0 commented 2 years ago

@petermetz

Refactoring to reduce redundancy is acceptable if functionalities provided by existing modules (e.g. monitoring functionality provided by startMonitor/stopMonitor API in cactus-cmd-socketio-server) are not removed.

In particular, the following functionalities described in the API table are needed.

takeutak commented 2 years ago

@petermetz Thanks for @izuru0's comment about the above API. Please let me add more information.

We also need to the functionality of posting transactions and getting transaction information and status described as the following whitepaper

The current codes are here

For example, these are called by the examples as the following:

The above are only part of those functionalities and I will add the other functionalities we already developed on this comment later. After finishing our preparations, I'm happy that we will discuss about the API design.

petermetz commented 2 years ago

@takeutak @izuru0 I responded to your emails.