forked from OmniLayer/spec
-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Specified Divident Addresses with a regular pay-out time interval.
- Loading branch information
1 parent
01e80af
commit 3049e5b
Showing
1 changed file
with
14 additions
and
0 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
3049e5b
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this should go into the product requirements document and be evaluated first, additionally there is no discussion of compliance and regulatory issues when implementing this
https://github.com/mastercoin-MSC/Master_Protocol_Product_Requirements
3049e5b
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd like to add a potential attack vector concern Peter Todd raised with pay-all (dividends). Every TX of this nature requires updating the balances of hundreds, thousands, or even millions of addresses. This makes each command potentially extremely computationally expensive. Somebody flooding the network with pay-all commands (or payments to a dispensary address such as this) could effectively bring the network down.
Consequently, we need to include an anti-spam fee for any command which potentially affects a large number of balances, preferably proportional to the number of addresses affected.
3049e5b
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.