Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

failed trade #4857

Closed
yononima opened this issue Nov 26, 2020 · 11 comments
Closed

failed trade #4857

yononima opened this issue Nov 26, 2020 · 11 comments

Comments

@yononima
Copy link

Description

An error occurred when taking the offer.

An error occurred at task: CreateTakerFeeTx
Exception message: Insufficient money, missing 0.03586202 BTC

No funds have left your wallet yet.
Please try to restart your application and check your network connection to see if you can resolve the issue.Please try to restart your application and check your network connection to see if you can resolve the issue.

Version

Steps to reproduce

Expected behaviour

Actual behaviour

Screenshots

Device or machine

macbook pro, OS
macOS Big Sur

Additional info

bisq.log

@boring-cyborg
Copy link

boring-cyborg bot commented Nov 26, 2020

Thanks for opening your first issue here!

Be sure to follow the issue template. Your issue will be reviewed by a maintainer and labeled for further action.

@ghost
Copy link

ghost commented Nov 27, 2020

Documenting my investigation so far:

Nov-24 18:11  Upgraded to 1.5.0

Nov-26 @ 22:22 Bisq starts up; last seen block is height 658809
Nov-26 @ 22:26 user sends OfferAvailabilityRequest for offer PR9AOEBS
Bisq wallet is empty.  User funds trade directly from external wallet.
Nov-26 @ 22:32 user funds trade wallet + 0.03804608 BTC
Nov-26 @ 22:34 user funds trade wallet + 0.002 BTC
Nov-26 @ 22:34 user sends OfferAvailabilityRequest for offer PR9AOEBS
Nov-26 @ 22:35 user sends OfferAvailabilityRequest for offer PR9AOEBS
Nov-26 @ 22:35 user takes offer but fails in CreateTakerFeeTx
    there is an exception thrown from BitcoinJ::completeTx -> presumably line 4170 within call to calculateFee
    Insufficient money,  missing 0.03586202 BTC
Nov-26 @ 22:41 block 658810 is published
  • it looks like the first UTXO was not considered for funding, why?
  • there seems to be one extra call to OfferAvailabilityRequest which I don't see when trying to reproduce this.

@yononima if you can provide any extra clues as to what you did to make this happen, it might help.

@chimp1984
Copy link
Contributor

@yononima Can you PM me (on keybase, same username) all logs files and a screenshot of the trade details and all tx Ids?
I assume the funds have been spent because for some reason the "reserved for offer" flag got removed so once one took the offer the reserved funds have not been on that address anymore. I saw similar cases but du lack of log could not find what causes it.

@chimp1984
Copy link
Contributor

Did you use BTC or BSQ as trade fee?

@yononima
Copy link
Author

yononima commented Dec 8, 2020

I used BSQ.

I will send you the log files the soonest possible.

The error occurred when i sent funds from a external wallet to fund tx security. When funds transferred, i couldn't take the offer before the btc amount was 1 satoshi less. So i had to put more funds in order to proceed with the tx. When i added some more satoshi and tried to proceed with the tx, i got the error.

I couldn't understand why it was 1 satoshi less, as i scanned the QR for funding.

@chimp1984
Copy link
Contributor

The 1 sat diff seems like a bug. thanks for your description was prob. some not well tested edge case. if we see the tx details we can see if the received funds was exact the requested ones. maybe the fee estimation was called again and the required funds changed (must not be the case). though if u cancel and take again same offer that is expected and required funds can be different as the fee is newly caclulated. I assume there is a bug that you got a new fee update, and while funding the trade the required funds got recalculated and failed then.

@chimp1984
Copy link
Contributor

I tested with updating the fee frequently while creating or taking offers with/without external funding (also with multiple funding txs) but could not reporduce the bug. Remaining therory is that the fee estimation code has some issue with rounding, but have not looked into that yet. But all in call it seems to me a rare edge case bug not releated to the other more frequent bugs we got reported since 1.4.2.

@yononima
Copy link
Author

yononima commented Dec 8, 2020

image

image

@yononima
Copy link
Author

yononima commented Dec 8, 2020

https://mempool.space/address/bc1qe705fdsd99jt03mv4x605txycnnfjtvl9h6fj6

"funds needed" and the tx above have 1 satoshi difference.

@stale
Copy link

stale bot commented Jun 11, 2021

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the was:dropped label Jun 11, 2021
@stale
Copy link

stale bot commented Jun 22, 2021

This issue has been automatically closed because of inactivity. Feel free to reopen it if you think it is still relevant.

@stale stale bot closed this as completed Jun 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants