Bitcoinjs-lib: Default TransactionBuilder Transaction version
Bitcoin Core defaults to 0x02.
What should we do?
dcousens
All 6 comments
We don't yet support signing CSV or stuff where version 2 is required (BIP112/BIP68), so we could save it until then.
afk11
on 5 Sep 2017
I think version 2 defaulting in Bitcoin Core is to help prevent fingerprinting of transactions to track lightning usage etc.
I say we should default to 2 to assist in obfuscation.
dabura667
on 5 Sep 2017
@dabura667 Also a fair point, but our transactions don't set a locktime, so are distinguishable from Bitcoin Core :)
afk11
on 5 Sep 2017
I say we should default to 2 to assist in obfuscation.
My reasoning, but there is a tipping point. I don't know what that it is right now. Stats needed.
@afk11 speak for yourself, but I set locktime :)
dcousens
on 5 Sep 2017
Assigned to 4.0.0
dcousens
on 19 Oct 2017
Closing in favour of https://github.com/bitcoinjs/bitcoinjs-lib/issues/957
dcousens
on 10 Dec 2017
Related issues
Mr-Mondragon
路
3Comments
askucher
路
3Comments
dcousens
路
3Comments
coingeek
路
4Comments
namnv04
路
3Comments
Most helpful comment
We don't yet support signing CSV or stuff where version 2 is required (BIP112/BIP68), so we could save it until then.