11215521440

Committed 07 Oct 2024 12:30PM UTC coverage: 64.696% (-0.02%) from 64.716%

Build # 11215521440

Build Type

Pull #14754

github

Committed by

web-flow

Commit Message

Merge 15587ca62 into 3fe9be026

Pull Request Pull Request #14754: rec: make sure the record cache has sane paramers for each unit test

Run Details

37077 of 88054 branches covered (42.11%)

Branch coverage included in aggregate %.

4 of 4 new or added lines in 3 files covered. (100.0%)

39 existing lines in 11 files now uncovered.

124974 of 162428 relevant lines covered (76.94%)

4334487.28 hits per line

Source File
Press 'n' to go to next uncovered line, 'b' for previous

63.49

/pdns/recursordist/rec-tcpout.cc

/*
 * This file is part of PowerDNS or dnsdist.
 * Copyright -- PowerDNS.COM B.V. and its contributors
 *
 * This program is free software; you can redistribute it and/or modify
 * it under the terms of version 2 of the GNU General Public License as
 * published by the Free Software Foundation.
 *
 * In addition, for the avoidance of any doubt, permission is granted to
 * link this program with OpenSSL and to (re)distribute the binaries
 * produced as the result of such linking.
 *
 * This program is distributed in the hope that it will be useful,
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 * GNU General Public License for more details.
 *
 * You should have received a copy of the GNU General Public License
 * along with this program; if not, write to the Free Software
 * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
 */

#include "rec-tcpout.hh"

// This line from /usr/include/openssl/ssl2.h: # define CERT char
// throws dnsrecords.hh off the rails.
#undef CERT

#include "syncres.hh"

timeval TCPOutConnectionManager::s_maxIdleTime;
size_t TCPOutConnectionManager::s_maxQueries;
size_t TCPOutConnectionManager::s_maxIdlePerAuth;
size_t TCPOutConnectionManager::s_maxIdlePerThread;

void TCPOutConnectionManager::cleanup(const struct timeval& now)
{
  if (s_maxIdleTime.tv_sec == 0 && s_maxIdleTime.tv_usec == 0) {
    // no maximum idle time
    return;
  }

  for (auto it = d_idle_connections.begin(); it != d_idle_connections.end();) {
    timeval idle = now - it->second.d_last_used;
    if (s_maxIdleTime < idle) {
      it = d_idle_connections.erase(it);
    }
    else {
      ++it;
    }
  }
}

void TCPOutConnectionManager::store(const struct timeval& now, const ComboAddress& ip, Connection&& connection)
{
  ++connection.d_numqueries;
  if (s_maxQueries > 0 && connection.d_numqueries >= s_maxQueries) {
    return;
  }

  if (d_idle_connections.size() >= s_maxIdlePerThread || d_idle_connections.count(ip) >= s_maxIdlePerAuth) {
    cleanup(now);
  }

  if (d_idle_connections.size() >= s_maxIdlePerThread) {
    return;
  }
  if (d_idle_connections.count(ip) >= s_maxIdlePerAuth) {
    return;
  }

  gettimeofday(&connection.d_last_used, nullptr);
  d_idle_connections.emplace(ip, std::move(connection));
}

TCPOutConnectionManager::Connection TCPOutConnectionManager::get(const ComboAddress& ip)
{
  if (d_idle_connections.count(ip) > 0) {
    auto h = d_idle_connections.extract(ip);
    return h.mapped();
  }
  return Connection{};
}

uint64_t getCurrentIdleTCPConnections()
{
  return broadcastAccFunction<uint64_t>([] { return t_tcp_manager.getSize(); });
}

1	/*
2	* This file is part of PowerDNS or dnsdist.
3	* Copyright -- PowerDNS.COM B.V. and its contributors
4	*
5	* This program is free software; you can redistribute it and/or modify
6	* it under the terms of version 2 of the GNU General Public License as
7	* published by the Free Software Foundation.
8	*
9	* In addition, for the avoidance of any doubt, permission is granted to
10	* link this program with OpenSSL and to (re)distribute the binaries
11	* produced as the result of such linking.
12	*
13	* This program is distributed in the hope that it will be useful,
14	* but WITHOUT ANY WARRANTY; without even the implied warranty of
15	* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
16	* GNU General Public License for more details.
17	*
18	* You should have received a copy of the GNU General Public License
19	* along with this program; if not, write to the Free Software
20	* Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
21	*/
22
23	#include "rec-tcpout.hh"
24
25	// This line from /usr/include/openssl/ssl2.h: # define CERT char
26	// throws dnsrecords.hh off the rails.
27	#undef CERT
28
29	#include "syncres.hh"
30
31	timeval TCPOutConnectionManager::s_maxIdleTime;
32	size_t TCPOutConnectionManager::s_maxQueries;
33	size_t TCPOutConnectionManager::s_maxIdlePerAuth;
34	size_t TCPOutConnectionManager::s_maxIdlePerThread;
35
36	void TCPOutConnectionManager::cleanup(const struct timeval& now)
37	{	756✔
38	if (s_maxIdleTime.tv_sec == 0 && s_maxIdleTime.tv_usec == 0) {	756!
39	// no maximum idle time
40	return;	×
41	}	×
42
43	for (auto it = d_idle_connections.begin(); it != d_idle_connections.end();) {	757✔
44	timeval idle = now - it->second.d_last_used;	1✔
45	if (s_maxIdleTime < idle) {	1!
UNCOV 46	it = d_idle_connections.erase(it);	×
UNCOV 47	}	×
48	else {	1✔
49	++it;	1✔
50	}	1✔
51	}	1✔
52	}	756✔
53
54	void TCPOutConnectionManager::store(const struct timeval& now, const ComboAddress& ip, Connection&& connection)
55	{	12✔
56	++connection.d_numqueries;	12✔
57	if (s_maxQueries > 0 && connection.d_numqueries >= s_maxQueries) {	12!
58	return;	×
59	}	×
60
61	if (d_idle_connections.size() >= s_maxIdlePerThread \|\| d_idle_connections.count(ip) >= s_maxIdlePerAuth) {	12!
62	cleanup(now);	×
63	}	×
64
65	if (d_idle_connections.size() >= s_maxIdlePerThread) {	12!
66	return;	×
67	}	×
68	if (d_idle_connections.count(ip) >= s_maxIdlePerAuth) {	12!
69	return;	×
70	}	×
71
72	gettimeofday(&connection.d_last_used, nullptr);	12✔
73	d_idle_connections.emplace(ip, std::move(connection));	12✔
74	}	12✔
75
76	TCPOutConnectionManager::Connection TCPOutConnectionManager::get(const ComboAddress& ip)
77	{	12✔
78	if (d_idle_connections.count(ip) > 0) {	12✔
79	auto h = d_idle_connections.extract(ip);	4✔
80	return h.mapped();	4✔
81	}	4✔
82	return Connection{};	8✔
83	}	12✔
84
85	uint64_t getCurrentIdleTCPConnections()
86	{	155✔
87	return broadcastAccFunction<uint64_t>([] { return t_tcp_manager.getSize(); });	854✔
88	}	155✔

PowerDNS / pdns / 11215521440

Source File Press 'n' to go to next uncovered line, 'b' for previous

Source File
Press 'n' to go to next uncovered line, 'b' for previous