OriginTrail / OT-RFC-repository

14 stars 3 forks source link

Discussion regarding the Annex of RFC 16 #39

Open NZT48 opened 11 months ago

NZT48 commented 11 months ago

Hi Tracers,

We're excited to share updates to RFC 16, focusing on bridges implementation for the OriginTrail Parachain. These updates provide latest information about Phase 1 and Phase 2 of the parachain bridges.

You can review the changes and gain more information by accessing the updated RFC through this link. Additionally, we welcome your feedback and comments on the proposal, which you can provide by opening a PR or reviewing the markdown file.

As always your feedback and comments are much appreciated.

Trace on!

Valcyclovir commented 11 months ago

Context: The introduction of Moonbeam TRAC and xcOTP as per RFC-16 has been going smoothly. The Wormhole bridge to bridge ERC-20 TRAC to moonbeamTRAC is live and working, and the xcOTP <-> moonbeamTRAC pair on Stellaswap seems to be working as well. As per the RFC, xcTRAC is well protected from moonbeamTRAC risks.

Issue: OTP can now be xcm transfered to moonbeam and be traded against moonbeamTRAC. In the event that moonbeamTRAC suffers an attack, the xcOTP <-> moonbeamTRAC pool will become compromised.

Objective:

Suggestion: Implement a xcOTP <-> xcTRAC pool instead of the current xcOTP <-> moonbeamTRAC pool. MoonbeamTRAC should only have 1 vector of attack rather than 2. The end user who bridges ERC-20 TRAC to moonbeam through Wormhole (or any other 3rd party bridge) can trade moonbeamTRAC for xcTRAC, and then exchange xcTRAC for some xcOTP. The number of steps for the end user remain the same, as the user would need to trade moonbeamTRAC for xcOTP and xcTRAC in any case. If moonbeamTRAC gets compromised, only 1 pool will be affected instead of both moonbeamTRAC <-> xcTRAC and moonbeamTRAC <-> xcOTP. The proposed solution is to implement a xcOTP <-> xcTRAC pool rather than the current xcOTP <-> moonbeamTRAC pool, and to keep moonbeamTRAC as isolated as possible, only allowing it to be exchanged against xcTRAC in order to mitigate risks.