Revert "Added SysConfig Setting to configure timeout for DNS resolver…

… queries. (#390)" This reverts commit 827f68c.
znuny · Nov 8, 2023 · 77fcf18 · 77fcf18
1 parent ec51ec0
commit 77fcf18
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 10 deletions.
diff --git a/Kernel/Config/Files/XML/Framework.xml b/Kernel/Config/Files/XML/Framework.xml
@@ -554,13 +554,6 @@
             <Item ValueType="String" ValueRegex="">ns.example.com</Item>
         </Value>
     </Setting>
-    <Setting Name="ResolverTimeout" Required="0" Valid="0">
-        <Description Translatable="1">Sets the timeout (in seconds) for resolver DNS queries (i.e. during checking MX record if CheckMXRecord is enabled to protect against long agent UI responses in case of very slow DNS queries).</Description>
-        <Navigation>Core::Email</Navigation>
-        <Value>
-            <Item ValueType="String" ValueRegex="">3</Item>
-        </Value>
-    </Setting>
     <Setting Name="CheckEmailAddresses" Required="1" Valid="1">
         <Description Translatable="1">Makes the application check the syntax of email addresses.</Description>
         <Navigation>Core::Email</Navigation>

diff --git a/Kernel/System/CheckItem.pm b/Kernel/System/CheckItem.pm
@@ -147,9 +147,8 @@ sub CheckEmail {
         if ($Resolver) {
 
             # it's no fun to have this hanging in the web interface
-            my $ResolverTimeout = $ConfigObject->Get('ResolverTimeout') // 3;
-            $Resolver->tcp_timeout($ResolverTimeout);
-            $Resolver->udp_timeout($ResolverTimeout);
+            $Resolver->tcp_timeout(3);
+            $Resolver->udp_timeout(3);
 
             # check if we need to use a specific name server
             my $Nameserver = $ConfigObject->Get('CheckMXRecord::Nameserver');