mirror of
https://github.com/basicswap/basicswap.git
synced 2025-01-03 17:29:26 +00:00
Add bid request accepted state.
This commit is contained in:
parent
303499fc6f
commit
705ac2c6fc
4 changed files with 43 additions and 43 deletions
|
@ -294,9 +294,9 @@ class BasicSwap(BaseApp):
|
|||
self.network_pubkey = self.settings['network_pubkey']
|
||||
self.network_addr = pubkeyToAddress(chainparams[Coins.PART][self.chain]['pubkey_address'], bytes.fromhex(self.network_pubkey))
|
||||
|
||||
self.db_echo = self.settings.get('db_echo', False)
|
||||
self.sqlite_file = os.path.join(self.data_dir, 'db{}.sqlite'.format('' if self.chain == 'mainnet' else ('_' + self.chain)))
|
||||
db_exists = os.path.exists(self.sqlite_file)
|
||||
self.db_echo: bool = self.settings.get('db_echo', False)
|
||||
self.sqlite_file: str = os.path.join(self.data_dir, 'db{}.sqlite'.format('' if self.chain == 'mainnet' else ('_' + self.chain)))
|
||||
db_exists: bool = os.path.exists(self.sqlite_file)
|
||||
|
||||
# HACK: create_all hangs when using tox, unless create_engine is called with echo=True
|
||||
if not db_exists:
|
||||
|
@ -1143,8 +1143,12 @@ class BasicSwap(BaseApp):
|
|||
def validateSwapType(self, coin_from, coin_to, swap_type):
|
||||
if (coin_from in self.adaptor_swap_only_coins or coin_to in self.adaptor_swap_only_coins) and swap_type != SwapTypes.XMR_SWAP:
|
||||
raise ValueError('Invalid swap type for: {} -> {}'.format(coin_from.name, coin_to.name))
|
||||
if (coin_from in self.secret_hash_swap_only_coins or coin_to in self.secret_hash_swap_only_coins) and swap_type == SwapTypes.XMR_SWAP:
|
||||
raise ValueError('Invalid swap type for: {} -> {}'.format(coin_from.name, coin_to.name))
|
||||
|
||||
if swap_type == SwapTypes.XMR_SWAP:
|
||||
if (coin_from in self.secret_hash_swap_only_coins or coin_to in self.secret_hash_swap_only_coins):
|
||||
raise ValueError('Invalid swap type for: {} -> {}'.format(coin_from.name, coin_to.name))
|
||||
if (coin_from in self.scriptless_coins and coin_to in self.scriptless_coins):
|
||||
raise ValueError('Invalid swap type for: {} -> {}'.format(coin_from.name, coin_to.name))
|
||||
|
||||
def notify(self, event_type, event_data, session=None) -> None:
|
||||
show_event = event_type not in self._disabled_notification_types
|
||||
|
@ -2780,7 +2784,7 @@ class BasicSwap(BaseApp):
|
|||
if ci_to.curve_type() == Curves.ed25519:
|
||||
self.sendXmrSplitMessages(XmrSplitMsgTypes.BID_ACCEPT, addr_from, addr_to, xmr_swap.bid_id, xmr_swap.kbsl_dleag, msg_valid, bid_msg_ids)
|
||||
|
||||
bid.setState(BidStates.BID_ACCEPTED)
|
||||
bid.setState(BidStates.BID_ACCEPTED) # ADS
|
||||
|
||||
session = self.openSession()
|
||||
try:
|
||||
|
@ -2862,18 +2866,19 @@ class BasicSwap(BaseApp):
|
|||
addr_from: str = offer.addr_from
|
||||
addr_to: str = bid.bid_addr
|
||||
msg_valid: int = self.getAcceptBidMsgValidTime(bid)
|
||||
bid_accept_id = self.sendSmsg(addr_from, addr_to, payload_hex, msg_valid)
|
||||
|
||||
bid_msg_ids = {}
|
||||
bid_msg_ids[0] = self.sendSmsg(addr_from, addr_to, payload_hex, msg_valid)
|
||||
|
||||
if ci_to.curve_type() == Curves.ed25519:
|
||||
self.sendXmrSplitMessages(XmrSplitMsgTypes.BID, addr_from, addr_to, xmr_swap.bid_id, xmr_swap.kbsf_dleag, msg_valid, bid_msg_ids)
|
||||
|
||||
bid.setState(BidStates.BID_REQUEST_ACCEPTED)
|
||||
|
||||
session = self.openSession()
|
||||
try:
|
||||
self.addMessageLink(Concepts.BID, bid_id, MessageTypes.ADS_BID_ACCEPT_FL, bid_accept_id, session=session)
|
||||
for k, msg_id in bid_msg_ids.items():
|
||||
self.addMessageLink(Concepts.BID, bid_id, MessageTypes.BID, msg_id, msg_sequence=k, session=session)
|
||||
self.log.info('Sent ADS_BID_ACCEPT_FL %s', bid_accept_id.hex())
|
||||
self.addMessageLink(Concepts.BID, bid_id, MessageTypes.ADS_BID_ACCEPT_FL, msg_id, msg_sequence=k, session=session)
|
||||
self.log.info('Sent ADS_BID_ACCEPT_FL %s', bid_msg_ids[0].hex())
|
||||
self.saveBidInSession(bid_id, bid, session, xmr_swap=xmr_swap)
|
||||
finally:
|
||||
self.closeSession(session)
|
||||
|
@ -4615,7 +4620,7 @@ class BasicSwap(BaseApp):
|
|||
bid.setState(BidStates.BID_RECEIVED)
|
||||
|
||||
self.saveBid(bid_id, bid)
|
||||
self.notify(NT.BID_RECEIVED, {'type': 'atomic', 'bid_id': bid_id.hex(), 'offer_id': bid_data.offer_msg_id.hex()})
|
||||
self.notify(NT.BID_RECEIVED, {'type': 'secrethash', 'bid_id': bid_id.hex(), 'offer_id': bid_data.offer_msg_id.hex()})
|
||||
|
||||
if self.shouldAutoAcceptBid(offer, bid):
|
||||
delay = random.randrange(self.min_delay_event, self.max_delay_event)
|
||||
|
@ -4761,7 +4766,7 @@ class BasicSwap(BaseApp):
|
|||
ensure(ci_from.verifyPubkey(xmr_swap.pkaf), 'Invalid pubkey, pkaf')
|
||||
|
||||
if not reverse_bid: # notify already ran in processADSBidReversed
|
||||
self.notify(NT.BID_RECEIVED, {'type': 'xmr', 'bid_id': bid.bid_id.hex(), 'offer_id': bid.offer_id.hex()}, session)
|
||||
self.notify(NT.BID_RECEIVED, {'type': 'ads', 'bid_id': bid.bid_id.hex(), 'offer_id': bid.offer_id.hex()}, session)
|
||||
|
||||
bid.setState(BidStates.BID_RECEIVED)
|
||||
|
||||
|
@ -4829,7 +4834,7 @@ class BasicSwap(BaseApp):
|
|||
if xmr_swap.pkal == xmr_swap.pkaf:
|
||||
raise ValueError('Duplicate script spend pubkey.')
|
||||
|
||||
bid.setState(BidStates.BID_ACCEPTED) # XMR
|
||||
bid.setState(BidStates.BID_ACCEPTED) # ADS
|
||||
self.saveBidInSession(bid.bid_id, bid, session, xmr_swap)
|
||||
self.notify(NT.BID_ACCEPTED, {'bid_id': bid.bid_id.hex()}, session)
|
||||
|
||||
|
@ -5000,7 +5005,7 @@ class BasicSwap(BaseApp):
|
|||
v = ci_from.verifyTxSig(xmr_swap.a_lock_refund_tx, xmr_swap.al_lock_refund_tx_sig, xmr_swap.pkal, 0, xmr_swap.a_lock_tx_script, prevout_amount)
|
||||
ensure(v, 'Invalid coin A lock refund tx leader sig')
|
||||
|
||||
allowed_states = [BidStates.BID_SENT, BidStates.BID_RECEIVED]
|
||||
allowed_states = [BidStates.BID_SENT, BidStates.BID_RECEIVED, BidStates.BID_REQUEST_ACCEPTED]
|
||||
if bid.was_sent and offer.was_sent:
|
||||
allowed_states.append(BidStates.BID_ACCEPTED) # TODO: Split BID_ACCEPTED into received and sent
|
||||
ensure(bid.state in allowed_states, 'Invalid state for bid {}'.format(bid.state))
|
||||
|
@ -5632,6 +5637,7 @@ class BasicSwap(BaseApp):
|
|||
|
||||
# Validate data
|
||||
ensure(len(msg_data.msg_id) == 28, 'Bad msg_id length')
|
||||
self.log.debug('for bid %s', msg_data.msg_id.hex())
|
||||
|
||||
# TODO: Wait for bid msg to arrive first
|
||||
|
||||
|
@ -5783,7 +5789,7 @@ class BasicSwap(BaseApp):
|
|||
|
||||
try:
|
||||
session = self.openSession()
|
||||
self.notify(NT.BID_RECEIVED, {'type': 'as_reversed', 'bid_id': bid.bid_id.hex(), 'offer_id': bid.offer_id.hex()}, session)
|
||||
self.notify(NT.BID_RECEIVED, {'type': 'ads_reversed', 'bid_id': bid.bid_id.hex(), 'offer_id': bid.offer_id.hex()}, session)
|
||||
|
||||
options = {'reverse_bid': True, 'bid_rate': bid_data.rate}
|
||||
if self.shouldAutoAcceptBid(offer, bid, session, options=options):
|
||||
|
|
|
@ -102,6 +102,7 @@ class BidStates(IntEnum):
|
|||
XMR_SWAP_MSG_SCRIPT_LOCK_TX_SIGS = 27 # XmrBidLockTxSigsMessage
|
||||
XMR_SWAP_MSG_SCRIPT_LOCK_SPEND_TX = 28 # XmrBidLockSpendTxMessage
|
||||
BID_REQUEST_SENT = 29
|
||||
BID_REQUEST_ACCEPTED = 30
|
||||
|
||||
|
||||
class TxStates(IntEnum):
|
||||
|
@ -303,6 +304,8 @@ def strBidState(state):
|
|||
return 'Exchanged script lock spend tx msg'
|
||||
if state == BidStates.BID_REQUEST_SENT:
|
||||
return 'Request sent'
|
||||
if state == BidStates.BID_REQUEST_ACCEPTED:
|
||||
return 'Request accepted'
|
||||
|
||||
return 'Unknown' + ' ' + str(state)
|
||||
|
||||
|
@ -494,29 +497,20 @@ inactive_states = [BidStates.SWAP_COMPLETED, BidStates.BID_ERROR, BidStates.BID_
|
|||
def isActiveBidState(state):
|
||||
if state >= BidStates.BID_ACCEPTED and state < BidStates.SWAP_COMPLETED:
|
||||
return True
|
||||
if state == BidStates.SWAP_DELAYING:
|
||||
return True
|
||||
if state == BidStates.XMR_SWAP_HAVE_SCRIPT_COIN_SPEND_TX:
|
||||
return True
|
||||
if state == BidStates.XMR_SWAP_SCRIPT_COIN_LOCKED:
|
||||
return True
|
||||
if state == BidStates.XMR_SWAP_NOSCRIPT_COIN_LOCKED:
|
||||
return True
|
||||
if state == BidStates.XMR_SWAP_LOCK_RELEASED:
|
||||
return True
|
||||
if state == BidStates.XMR_SWAP_NOSCRIPT_TX_REDEEMED:
|
||||
return True
|
||||
if state == BidStates.XMR_SWAP_SCRIPT_TX_REDEEMED:
|
||||
return True
|
||||
if state == BidStates.XMR_SWAP_SCRIPT_TX_PREREFUND:
|
||||
return True
|
||||
if state == BidStates.XMR_SWAP_MSG_SCRIPT_LOCK_TX_SIGS:
|
||||
return True
|
||||
if state == BidStates.XMR_SWAP_MSG_SCRIPT_LOCK_SPEND_TX:
|
||||
return True
|
||||
if state == BidStates.XMR_SWAP_FAILED:
|
||||
return True
|
||||
return False
|
||||
return state in (
|
||||
BidStates.SWAP_DELAYING,
|
||||
BidStates.XMR_SWAP_HAVE_SCRIPT_COIN_SPEND_TX,
|
||||
BidStates.XMR_SWAP_SCRIPT_COIN_LOCKED,
|
||||
BidStates.XMR_SWAP_NOSCRIPT_COIN_LOCKED,
|
||||
BidStates.XMR_SWAP_LOCK_RELEASED,
|
||||
BidStates.XMR_SWAP_NOSCRIPT_TX_REDEEMED,
|
||||
BidStates.XMR_SWAP_SCRIPT_TX_REDEEMED,
|
||||
BidStates.XMR_SWAP_SCRIPT_TX_PREREFUND,
|
||||
BidStates.XMR_SWAP_MSG_SCRIPT_LOCK_TX_SIGS,
|
||||
BidStates.XMR_SWAP_MSG_SCRIPT_LOCK_SPEND_TX,
|
||||
BidStates.XMR_SWAP_FAILED,
|
||||
BidStates.BID_REQUEST_ACCEPTED,
|
||||
)
|
||||
|
||||
|
||||
def isErrorBidState(state):
|
||||
|
|
|
@ -81,14 +81,14 @@ def upgradeDatabaseData(self, data_version):
|
|||
swap_ended = isFinalBidState(state)
|
||||
session.execute('UPDATE bidstates SET in_error = :in_error, swap_failed = :swap_failed, swap_ended = :swap_ended WHERE state_id = :state_id', {'in_error': in_error, 'swap_failed': swap_failed, 'swap_ended': swap_ended, 'state_id': int(state)})
|
||||
if data_version > 0 and data_version < 4:
|
||||
for state in (BidStates.BID_REQUEST_SENT, ):
|
||||
for state in (BidStates.BID_REQUEST_SENT, BidStates.BID_REQUEST_ACCEPTED):
|
||||
session.add(BidState(
|
||||
active_ind=1,
|
||||
state_id=int(state),
|
||||
in_progress=isActiveBidState(state),
|
||||
in_error=isErrorBidState(state),
|
||||
swap_failed = isFailingBidState(state),
|
||||
swap_ended = isFinalBidState(state),
|
||||
swap_failed=isFailingBidState(state),
|
||||
swap_ended=isFinalBidState(state),
|
||||
label=strBidState(state),
|
||||
created_at=now))
|
||||
|
||||
|
|
|
@ -478,7 +478,7 @@ def js_generatenotification(self, url_split, post_string, is_json) -> bytes:
|
|||
elif r == 2:
|
||||
swap_client.notify(NT.BID_ACCEPTED, {'bid_id': random.randbytes(28).hex()})
|
||||
elif r == 3:
|
||||
swap_client.notify(NT.BID_RECEIVED, {'type': 'xmr', 'bid_id': random.randbytes(28).hex(), 'offer_id': random.randbytes(28).hex()})
|
||||
swap_client.notify(NT.BID_RECEIVED, {'type': 'ads', 'bid_id': random.randbytes(28).hex(), 'offer_id': random.randbytes(28).hex()})
|
||||
|
||||
return bytes(json.dumps({'type': r}), 'UTF-8')
|
||||
|
||||
|
|
Loading…
Reference in a new issue