Merge pull request #7860 from pieterlexis/dnsdist-rm-disable-tcp

dnsdist: disallow TCP disablement

diff --cc pdns/dnsdist-lua.cc
index bf235d557230718bd854a9131e58a37cc5e4baa6,3db0e822fc15ff2b57f843cc0452ad9aa3b7b0ec..aab963cf6675544865fb5d86c8f1dd9f4a836bcd
--- 1/pdns/dnsdist-lua.cc
--- 2/pdns/dnsdist-lua.cc
+++ b/pdns/dnsdist-lua.cc
@@@ -536,13 -529,10 +529,11 @@@ void setupLuaConfig(bool client
        ComboAddress loc(addr, 53);
          // only works pre-startup, so no sync necessary
          g_frontends.push_back(std::unique_ptr<ClientState>(new ClientState(loc, false, reusePort, tcpFastOpenQueueSize, interface, cpus)));
-         if (doTCP) {
-           g_frontends.push_back(std::unique_ptr<ClientState>(new ClientState(loc, true, reusePort, tcpFastOpenQueueSize, interface, cpus)));
-         }
+         g_frontends.push_back(std::unique_ptr<ClientState>(new ClientState(loc, true, reusePort, tcpFastOpenQueueSize, interface, cpus)));
        }
        catch(std::exception& e) {
 -      g_outputBuffer="Error: "+string(e.what())+"\n";
 +        g_outputBuffer="Error: "+string(e.what())+"\n";
 +        errlog("Error while trying to listen on %s: %s\n", addr, string(e.what()));
        }
      });
  

diff --cc pdns/dnsdistdist/docs/reference/config.rst
index 5f62a7b74e9e4e7a1b0eabaa5d3ebd1530b33f5d,bd94c1a922176168da3a22b59cb38b5be3a7fa9f..00b2293f8f58990fabd85dc7c05870a011bccd0d
--- 1/pdns/dnsdistdist/docs/reference/config.rst
--- 2/pdns/dnsdistdist/docs/reference/config.rst
+++ b/pdns/dnsdistdist/docs/reference/config.rst
@@@ -74,9 -77,9 +77,9 @@@ Listen Socket
  
    Options:
  
-   * ``doTCP=true``: bool - Also bind on TCP on ``address``.
+   * ``doTCP=true``: bool - Also bind on TCP on ``address``. Removed in 1.4.0.
    * ``reusePort=false``: bool - Set the ``SO_REUSEPORT`` socket option.
 -  * ``tcpFastOpenSize=0``: int - Set the TCP Fast Open queue size, enabling TCP Fast Open when available and the value is larger than 0.
 +  * ``tcpFastOpenQueueSize=0``: int - Set the TCP Fast Open queue size, enabling TCP Fast Open when available and the value is larger than 0.
    * ``interface=""``: str - Set the network interface to use.
    * ``cpus={}``: table - Set the CPU affinity for this listener thread, asking the scheduler to run it on a single CPU id, or a set of CPU ids. This parameter is only available if the OS provides the pthread_setaffinity_np() function.
  

diff --cc pdns/dnsdistdist/docs/reference/dnscrypt.rst
index e5a04371b3c9d5a9e1452c2bfbcda0eb0ff15200,68a7703b9d32c71e9362c16ea5e255cc30c6d361..d25c6cfce888d57acf0b9727a807e88648b1560d
--- 1/pdns/dnsdistdist/docs/reference/dnscrypt.rst
--- 2/pdns/dnsdistdist/docs/reference/dnscrypt.rst
+++ b/pdns/dnsdistdist/docs/reference/dnscrypt.rst
@@@ -16,9 -19,9 +19,9 @@@ DNSCrypt objects and function
  
    Options:
  
-   * ``doTCP=true``: bool - Also bind on TCP on ``address``.
+   * ``doTCP=true``: bool - Also bind on TCP on ``address``, removed in 1.4.0.
    * ``reusePort=false``: bool - Set the ``SO_REUSEPORT`` socket option.
 -  * ``tcpFastOpenSize=0``: int - Set the TCP Fast Open queue size, enabling TCP Fast Open when available and the value is larger than 0
 +  * ``tcpFastOpenQueueSize=0``: int - Set the TCP Fast Open queue size, enabling TCP Fast Open when available and the value is larger than 0
    * ``interface=""``: str - Sets the network interface to use
    * ``cpus={}``: table - Set the CPU affinity for this listener thread, asking the scheduler to run it on a single CPU id, or a set of CPU ids. This parameter is only available if the OS provides the pthread_setaffinity_np() function.