Check interruptNet during dnsseed lookups #10215

1. 
   TheBlueMatt commented at 8:34 pm on April 14, 2017: member

   Since #9229 we no longer can break out of a DNS lookup midway through, and have to wait for libc to time out. In most places we check for interrupt immediately before/after, so this is not a huge deal, except in the dnsseed lookup thread. In that thread, we loop over all dnsseeds and will have to wait for each to timeout. This appears to be the cause of the long wait in #10210.

   Until upstream glibc fixes the issue (https://sourceware.org/bugzilla/show_bug.cgi?id=20874) with getaddrinfo_a or we move to libevent’s DNS lookup library, the best we can do is insert more agressive checks between lookups, which we do here.

2. 
   TheBlueMatt commented at 8:36 pm on April 14, 2017: member

   Should likely get backported for 0.14.2 (if we do one).
3. 
   theuni commented at 8:44 pm on April 14, 2017: member

   Hmm, I could’ve sworn we already did this.

   drop the (), then utACK :)

4. Check interruptNet during dnsseed lookups b2c9254205
5. TheBlueMatt force-pushed on Apr 14, 2017
6. theuni approved
7. 
   theuni commented at 9:36 pm on April 14, 2017: member

   utACK b2c9254205a59fa685d75470f73ad9397ecba708
8. MarcoFalke added the label Needs backport on Apr 14, 2017
9. MarcoFalke added this to the milestone 0.14.2 on Apr 14, 2017
10. gmaxwell approved
11. 
    gmaxwell commented at 5:47 am on April 15, 2017: contributor

    utACK.
12. MarcoFalke added the label P2P on Apr 15, 2017
13. 
    laanwj commented at 7:55 am on April 17, 2017: member

    utACK b2c9254
14. 
    sipa commented at 11:17 am on April 17, 2017: member

    utACK b2c9254205a59fa685d75470f73ad9397ecba708
15. sipa merged this on Apr 17, 2017
16. sipa closed this on Apr 17, 2017

    ---

17. sipa referenced this in commit a077a90da8 on Apr 17, 2017
18. laanwj removed the label Needs backport on May 31, 2017
19. codablock referenced this in commit ee3ff3552d on Jan 26, 2018
20. andvgal referenced this in commit 600c5a3869 on Jan 6, 2019
21. CryptoCentric referenced this in commit df7d94b9f8 on Feb 27, 2019
22. furszy referenced this in commit ecc46049f8 on Jan 13, 2021
23. DrahtBot locked this on Sep 8, 2021

Contributors
TheBlueMatt theuni gmaxwell laanwj sipa

Labels
P2P

Milestone
0.14.2