CESSProject / cess

An implementation of the distributed storage network supported by CESS LAB.
https://www.cess.cloud/
Apache License 2.0
297 stars 100 forks source link

The original `MinerInfo.peer_id` field of the storage miner is no longer applicable #404

Open 0xbillw opened 1 month ago

0xbillw commented 1 month ago

Cause

The current version of MinerInfo.peer_id represents the ID of the storage-miner P2P node. Due to the change of data transmission protocol for storage-miner, this field no longer has practical significance. On the other hand, the information used to represent the endpoint of the storage-miner borrows this field, resulting in improper expression of business semantics and making it difficult for future code maintenance.

Advices

  1. Deprecate the MinerInfo.peer_id field
  2. Add a field similar to endpoint to represent the API endpoint of the storage miner

Affect modules

  1. cess
  2. cess-miner
  3. cess-go-sdk
  4. other downside modules