Code Monkey home page Code Monkey logo

Comments (5)

turbolay avatar turbolay commented on June 20, 2024 2

In case there is enough change, RBF is really easy to implement -> Just reduce change

If there is not enough change, there are 4 possibilities:

  • Wallet has other private/same pocket coins bigger than some selected so swapping some selected coins with other could be enough to cover the fee -> only downside is potentially (more) change
  • Wallet has other private/same pocket coins smaller than the selected ones, some inputs can be added to cover for the extra fee -> it will increase size of the TX + potentially create (more) change
  • Amount sent can be reduced to cover for the extra fee -> best solution but causes issues for payments.
  • An extra non private output from another pocket -> degrades privacy, increases size of the TX and potentially create (more) change

In all cases it's probably better to just re-run the automatic coin selector but making sure that old TX will be replaced by the new one. One remaining question: If these are the only possible outcomes, should the user have an actionnable choice whether he wants to degrade his privacy by adding another pocket, have change, or reduce the amount?

from walletwasabi.

petertodd avatar petertodd commented on June 20, 2024 2

@turbolay

From the point of view of simplicity, I'd recommend that the original transaction inputs always be kept when RBF fee bumping. You don't want to accidentally create two distinct transactions, eg by selecting a different input to keep in different, successive, fee bumps. Better to lose a bit of privacy for the first version of this feature than accidentally create a double-payment bug.

from walletwasabi.

yahiheb avatar yahiheb commented on June 20, 2024 1

The effective fee rate calculation gets difficult if there is a chain of unconfirmed transactions prior to the users transaction.

This shouldn't be a problem if we are doing this for only single user transactions that are initiated from the users wallet (user has all inputs).

from walletwasabi.

MaxHillebrand avatar MaxHillebrand commented on June 20, 2024 1

the UX should warn the user about the privacy downsides [change output detection and potential additional non-private inputs]

from walletwasabi.

MaxHillebrand avatar MaxHillebrand commented on June 20, 2024

The effective fee rate calculation gets difficult if there is a chain of unconfirmed transactions prior to the users transaction.

from walletwasabi.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.