diff options
| author | lukehamburg <luke.hamburg@gmail.com> | 2016-02-26 23:31:01 -0500 |
|---|---|---|
| committer | Stephen Beaver <sbeaver@netgate.com> | 2016-02-29 11:50:58 -0500 |
| commit | f8a475f574e1a5b46c58f9fd7c7d2c5c7349f691 (patch) | |
| tree | a8d8d9e2e9d6ac9bd9d1bf309f3bf57af90b41ea /src/etc/inc/unbound.inc | |
| parent | b715db5d7396ddd5eb46f48d7d30cd9177777e02 (diff) | |
| download | pfsense-f8a475f574e1a5b46c58f9fd7c7d2c5c7349f691.zip pfsense-f8a475f574e1a5b46c58f9fd7c7d2c5c7349f691.tar.gz | |
Fix unbound.inc so zero value for *_num_tcp is honored
In the Unbound documentation, it is noted that setting "outgoing-num-tcp" to "0" disables TCP queries. In testing I noticed that although I could select & save a value of "0" in the Unbound Advanced Config, the actual value written to /var/unbound/unbound.conf was still the default "10". Apparently in PHP, a value of "0" still tests "false" for !empty(). Changing the test to isset() seems to fix the issue.
Diffstat (limited to 'src/etc/inc/unbound.inc')
| -rw-r--r-- | src/etc/inc/unbound.inc | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/etc/inc/unbound.inc b/src/etc/inc/unbound.inc index c716d1a..8a678a4 100644 --- a/src/etc/inc/unbound.inc +++ b/src/etc/inc/unbound.inc @@ -281,8 +281,8 @@ EOF; $harden_dnssec_stripped = isset($unboundcfg['dnssecstripped']) ? "yes" : "no"; $prefetch = isset($unboundcfg['prefetch']) ? "yes" : "no"; $prefetch_key = isset($unboundcfg['prefetchkey']) ? "yes" : "no"; - $outgoing_num_tcp = (!empty($unboundcfg['outgoing_num_tcp'])) ? $unboundcfg['outgoing_num_tcp'] : "10"; - $incoming_num_tcp = (!empty($unboundcfg['incoming_num_tcp'])) ? $unboundcfg['incoming_num_tcp'] : "10"; + $outgoing_num_tcp = isset($unboundcfg['outgoing_num_tcp']) ? $unboundcfg['outgoing_num_tcp'] : "10"; + $incoming_num_tcp = isset($unboundcfg['incoming_num_tcp']) ? $unboundcfg['incoming_num_tcp'] : "10"; $edns_buffer_size = (!empty($unboundcfg['edns_buffer_size'])) ? $unboundcfg['edns_buffer_size'] : "4096"; $num_queries_per_thread = (!empty($unboundcfg['num_queries_per_thread'])) ? $unboundcfg['num_queries_per_thread'] : "4096"; $jostle_timeout = (!empty($unboundcfg['jostle_timeout'])) ? $unboundcfg['jostle_timeout'] : "200"; |
