Author

Topic: JSON-RPC mass-pay (Read 1103 times)

legendary
Activity: 1596
Merit: 1091
March 02, 2011, 01:42:25 AM
#3
[...] though nodes should ignore all fees when relaying.

Such a policy increases the ability of third parties to force the entire network to relay garbage.

administrator
Activity: 5166
Merit: 12850
March 02, 2011, 01:04:37 AM
#2
That seems fair for a default fee rule, though nodes should ignore all fees when relaying.

When I complained about the introduction of that particular code on the forum, Satoshi asked me to stop talking about it due to a DoS risk. He seemed to think the risk was pretty serious. This makes me think there might be some hidden DoS vulnerability we don't know about. This was when clients were still waiting for a block number to change the rule, though, so it might have been just the transitioning that caused the vulnerability.

Quote from: satoshi
The thing you highlighted is a patch for a DoS
weakness.  I don't want to post an instruction manual for how to use the
weakness to DoS the previous versions.

I think most P2P networks, and websites for that matter, are vulnerable
to an endless number of DoS attacks.  The best we can realistically do
is limit the worst cases.

(I'm absolutely in favor of the feature -- I just want to mention Satoshi's view.)
legendary
Activity: 1652
Merit: 2216
Chief Scientist
March 01, 2011, 08:21:17 PM
#1
Add a mass-pay JSON-RPC method, provided that the user interface requires a TX fee parameter (NOTE: zero is a valid TX fee)

I've got a (private so far) patch that creates mass-pay transactions.  API is:
  sendmulti {address:amount,...} [minconf=1] [comment]

This code in CTransaction::AcceptToMemoryPool will need to change, too:
Code:
      if (GetSigOpCount() > 2 || nSize < 100)
        return error("AcceptToMemoryPool() : nonstandard transaction");
(a mass-pay transaction will have N+1 OP_CHECKSIGs in it, where N is the number of people being paid).

Replacing in the stock client with something like:
Code:
  if (GetSigOpCount() > 2) minFee += GetSigOpCount()*GetArgMoney("-masspayfee", CENT/100);
... seems like the right direction to go.

BUT: I think a higher priority is figuring out how to deal with fees for the other two send methods.
Jump to: