Stability for Flow

[caffeinum]

Stability for Flow

[caffeinum]

on hold

[caffeinum]

Есть предложение разделить Flow на две части:

• setup
• withdraw

Шаги убрать, вместо этого добавить список проверок, делать их можно в любом порядке. Это значит, почти весь код из Flow перенести в классы ***Swap. А Flow сведется к

// BTC2ETH
try {
  await btcSwap.setup()
  await ethSwap.check()
  await ethSwap.withdraw()
} catch () {
  await btcSwap.refund()
}

Например, список необходимых действий для обменов BTC<->ETH

ETHSwap
Sender	Recipient
lock funds	check contract balance
	withdraw

BTCSwap
Sender	Recipient
request sign	request script values
submit secret	check script
sync balance	verify script
lock funds	withdraw
send script values

[caffeinum]

All in all, state has two parts:

• informational flags
• crypto data

On exiting every step, Flow should have this data:

step	flags	data
setup	meSigned,participantSigned
sync balance	balanceEnough	balance
lock base	baseLocked	scriptValues, (secret), secretHash, lockedBalance, lockTxHash
lock seco	secondaryLocked	lockedBalance, lockTxHash
withdraw seco	secondaryWithdrawn	withdrawTxHash, (secret)
withdraw base	baseWithdrawn	withdrawTxHash

Also, for every step, there is a person, who decides step's finished.

step	responsible
setup	both
sync balance	both
lock base	secondaryOwner
lock seco	baseOwner
withdraw seco	secondaryOwner
withdraw base	baseOwner
finish	both

[caffeinum]

this.state = {
  step: 0,

  signTransactionHash: null,
  isSignFetching: false,
  isMeSigned: false,

  secretHash: null,
  btcScriptValues: null,

  btcScriptVerified: false,

  isBalanceFetching: false,
  isBalanceEnough: false,
  balance: null,

  btcScriptCreatingTransactionHash: null,
  ethSwapCreationTransactionHash: null,

  isEthContractFunded: false,

  secret: null,

  isEthWithdrawn: false,
  isBtcWithdrawn: false,

  refundTransactionHash: null,
  isRefunded: false,

  isFinished: false,
}

[0xZick]

полностью за, это как Игорь сделал EOS - BTC только ты хочешь более явно разделить

[noxonsu]

чеклист перенести в темплейты ишью и закрыть можно думаю

[caffeinum]

@noxonsu ишью о том, чтоб учесть эти все проблемы в архитектуре. Можно временно закрыть, наверное

[noxonsu]

перенести в вики тогда и закрыть

[caffeinum]

Это более общий вариант #4

[caffeinum]

Пока что мы пришли примерно к такому, если вкратце:

Вначале берем BTC2EOS Игоря, а дальше смотрим:

pull Handlers

Это круто, более того, они нужны везде, поэтому логично положить из в swap.swap/Room.js

push Handlers

По сути являются сообщениями, чаще всего это ответы на pull. Их можно сделать структурами данных без кода вовсе.

import { scriptValuesMessageGenerator } from '../btc/push'
room.sendMessage(scriptValuesMessageGenerator(state))

transaction Handlers

Тут самый сок. Это именно то, что отвечает прямо за действия.

Но почему бы не класть их напрямую в BtcSwap, EthSwap ? Это как раз действия с блокчейном.

Примеры идей:
caffeinum/swap.core@fd5925a

[noxonsu]

устарело