Make transactions larger than 100K non-standard #2273

pull gavinandresen wants to merge 1 commits into bitcoin:master from gavinandresen:txsize changing 3 files +11 −1
  1. gavinandresen commented at 10:17 pm on February 4, 2013: contributor

    Extremely large transactions with lots of inputs can cost the network almost as much to process as they cost the sender in fees.

    We would never create transactions larger than 100K big; this change makes transactions larger than 100K non-standard, so they are not relayed/mined by default. This is most important for miners that might create blocks larger than 250K big, who could be vulnerable to a make-your-blocks-so-expensive-to-verify-they-get-orphaned attack.

  2. gavinandresen commented at 10:25 pm on February 4, 2013: contributor

    FYI:

    In the last 1,000 blocks, there was exactly 1 transaction larger than 100,000 bytes: http://blockexplorer.com/tx/454f61695d3fb320cc538003b9c8ca1ba982eb7cd71abe8c9ecc02a1f843da1f

  3. gmaxwell commented at 10:28 pm on February 4, 2013: contributor
    ACK. … that >100k example is a nice utxo set cleaner, I’m sad to not see more of those!
  4. BitcoinPullTester commented at 2:07 am on February 5, 2013: none
    Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/0b377f4ee36d9468b63ec5c207f281b1f2003f49 for binaries and test log.
  5. sipa commented at 9:36 am on February 5, 2013: member
    ACK
  6. Make transactions larger than 100K non-standard
    Extremely large transactions with lots of inputs can cost the network
    almost as much to process as they cost the sender in fees.
    
    We would never create transactions larger than 100K big; this change
    makes transactions larger than 100K non-standard, so they are not
    relayed/mined by default. This is most important for miners that might
    create blocks larger than 250K big, who could be vulnerable to a
    make-your-blocks-so-expensive-to-verify-they-get-orphaned attack.
    41e1a0d766
  7. gavinandresen commented at 3:11 pm on February 5, 2013: contributor
    Updated to add a comment as suggested by Mike Hearn.
  8. gavinandresen referenced this in commit 1d0851eca8 on Feb 5, 2013
  9. gavinandresen merged this on Feb 5, 2013
  10. gavinandresen closed this on Feb 5, 2013

  11. gavinandresen deleted the branch on Nov 4, 2013
  12. laudney referenced this in commit 2a8c5f1823 on Mar 19, 2014
  13. DrahtBot locked this on Sep 8, 2021

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2024-12-26 21:12 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me