[qa] Add test for max number of entries in locator #13915

1. 
   MarcoFalke commented at 8:48 PM on August 8, 2018: member

   Should not be merged before #13907

2. MarcoFalke added the label Tests on Aug 8, 2018
3. 
   MarcoFalke commented at 8:48 PM on August 8, 2018: member

   Thanks for @jnewbery providing early feedback.

4. in test/functional/p2p_invalid_locator.py:26 in fa310aeb0f outdated

     21 | +
     22 | +        self.log.info('Test max locator size')
     23 | +        block_count = node.getblockcount()
     24 | +        for msg in [msg_getheaders(), msg_getblocks()]:
     25 | +            self.log.info('Wait for disconnect when sending {} hashes in locator'.format(MAX_LOCATOR_SZ + 1))
     26 | +            node.add_p2p_connection(P2PInterface())
   

   ---

   ---

   

   jnewbery commented at 9:03 PM on August 8, 2018:

   Sorry, I was unclear in my comment previously. We still need to call wait_for_verack(). What I meant was that we should move that call up into P2PInterface.peer_connect() (a bit like the current send_version argument).

5. 
   jnewbery commented at 9:05 PM on August 8, 2018: member

   One comment inline. Otherwise looks great.

6. 
   jnewbery commented at 9:44 PM on August 8, 2018: member

   Tested ACK fa310aeb0fb1840cf5197057fad6ed376e7c8ec3 ~(assuming #13916 goes in, otherwise I think node connection in this is racy)~

   EDIT: 13916 is merged. Unqualified tested ACK :grin:

7. 
   DrahtBot commented at 10:47 PM on August 9, 2018: member

   <!--e57a25ab6845829454e8d69fc972939a-->No more conflicts as of last run.

8. in test/functional/test_framework/messages.py:35 in fa310aeb0f outdated

     31 | @@ -32,6 +32,7 @@
     32 |  MY_RELAY = 1 # from version 70001 onwards, fRelay should be appended to version messages (BIP37)
     33 |  
     34 |  MAX_INV_SZ = 50000
     35 | +MAX_LOCATOR_SZ = 64
   

   ---

   ---

   

   MarcoFalke commented at 2:36 PM on August 10, 2018:

   Need to be adjusted before merge!

9. DrahtBot added the label Needs rebase on Aug 10, 2018
10. qa: Add p2p_invalid_locator test fa85c985ed
11. MarcoFalke force-pushed on Aug 10, 2018
12. 
    MarcoFalke commented at 6:10 PM on August 10, 2018: member

    Force pushed new version. Only change should be 64->101

13. MarcoFalke added this to the milestone 0.17.0 on Aug 10, 2018
14. 
    achow101 commented at 6:52 PM on August 10, 2018: member

    ACK fa85c985edbfd297aefb613445deab37c8769753

15. DrahtBot removed the label Needs rebase on Aug 10, 2018
16. 
    sipa commented at 10:37 PM on August 10, 2018: member

    Concept ACK

17. 
    gmaxwell commented at 10:01 AM on August 11, 2018: contributor

    ACK

18. MarcoFalke merged this on Aug 11, 2018
19. MarcoFalke closed this on Aug 11, 2018

    ---

20. MarcoFalke referenced this in commit a04888a075 on Aug 11, 2018
21. MarcoFalke deleted the branch on Aug 11, 2018
22. UdjinM6 referenced this in commit 7a4c891884 on Jul 4, 2021
23. UdjinM6 referenced this in commit c70e3cf126 on Jul 6, 2021
24. UdjinM6 referenced this in commit 290a23a9e4 on Jul 6, 2021
25. MarcoFalke locked this on Sep 8, 2021

Milestone
0.17.0