[bisq-network/bisq] Add irregular txType, add check for total balance, prevent proposal withhold attack (#2587)
sqrrm
notifications at github.com
Sun Mar 31 12:23:32 UTC 2019
sqrrm approved this pull request.
utACK
I think changing the name to burntBsq made it a lot easier to read and understand. Looks like a good solution.
> .mapToLong(merit -> {
try {
Issuance issuance = merit.getIssuance();
checkArgument(issuance.getIssuanceType() == IssuanceType.COMPENSATION,
"issuance must be of type COMPENSATION");
return getWeightedMeritAmount(issuance.getAmount(),
issuance.getChainHeight(),
- txChainHeight,
+ blindVoteTxHeight,
This seems like a good change, more consistent calculations. It also counts the merit at the time of voting which is what the user would expect.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/bisq-network/bisq/pull/2587#pullrequestreview-220859808
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.bisq.network/pipermail/bisq-github/attachments/20190331/0ff245af/attachment.html>
More information about the bisq-github
mailing list