Fix clang-tidy warnings

clang-tidy: pass by value

Signed-off-by: Rosen Penev <rosenp@gmail.com>

clang-tidy: use default member init

Signed-off-by: Rosen Penev <rosenp@gmail.com>

boost::replace_all > std::replace

Signed-off-by: Rosen Penev <rosenp@gmail.com>

clang-tidy: convert to range for loop

Found with modernize-loop-convert

Signed-off-by: Rosen Penev <rosenp@gmail.com>

Merge pull request #15067 from rgacogne/ddist-fix-coverity-yaml-2

dnsdist: Install python-yaml in the Coverity workflow

Merge pull request #15051 from miodvallat/pdnsutil_eulenspiegel

Boring pdnsutil cleanups

Merge pull request #14954 from neheb/mm

clang-tidy: use std::min/max

dnsdist: Install python-yaml in the Coverity workflow

Merge pull request #15066 from omoerbeek/tidy-credentials

Tidy credentials.??

Typo in comment

Co-authored-by: Remi Gacogne <github@coredump.fr>

Merge pull request #14581 from d-wibowo/direct-dnskey-signature

auth: added a new config for direct queries of dnskey signature

Tidy credentials.??

Merge pull request #15063 from omoerbeek/credentials-include

Include cstdint to get uint64_t

Merge pull request #15064 from rgacogne/ddist-fix-el-builds

dnsdist: Fix missing variant header that prevents building on EL8

clang-tidy

dnsdist: Fix missing variant header that prevents building on EL8

Merge pull request #15056 from omoerbeek/rec-old-settings-warn

rec: better phrased --enable-old-settings warning

Merge pull request #15061 from rgacogne/ddist-fix-coverity

dnsdist: Fix missing LLD dependency in the Coverity workflow

Include cstdint to get uint64_t

Should fix #15060

Merge pull request #15057 from omoerbeek/rec-coverity-20250120

rec: Coverity issues missed in last round, all severity "Low"

rec: better phrased --enable-old-settings warning

dnsdist: Fix missing LLD dependency in the Coverity workflow

rec: Coverity issues missed in last round, all severity "Low"

1587809 Variable copied when it could be moved
1587817 Variable copied when it could be moved
1587819 Variable copied when it could be moved

Address some clang-tidy complaints in pdnsutil.

Merge pull request #14969 from rgacogne/ddist-yaml-configuration-harder

dnsdist: Add a new, optional, YAML-based configuration format

Merge pull request #15050 from omoerbeek/rec-coverity-20240117

rec: New Coverity Variable copied when it could be moved cases

dnsdist: Fix broken refs and formatting issues in the YAML docs

rec: New Coverity Variable copied when it could be moved cases
Coverity:
1587816
1587811,12,13,14,17,19
1587810
1587809
1587808
1587807
1587806
1587805
1587804
1587801
1587798
1587796
1587795
1587792
1587791
1587790
1587789
1587788
1587786
1544953

dnsdist: Ad more missing description for YAML settings

Address coverity complaints in pdnsutil.

only look up direct-dnskey-signature in the config once

assert key_tag too

Merge pull request #15040 from omoerbeek/rec-ecs

EDNSSubnetOpts refactoring

Add a method to get the scoped subnet (aka Netmask) as suggested by @rgacogne

Merge pull request #15038 from omoerbeek/rec-record-sizes

rec: Compute size *estimate* for record cache allocated size

Merge pull request #15046 from rgacogne/ddist-better-error-msg-source

dnsdist: Better error when we cannot connect a backend socket

dnsdist: Better error when we cannot connect a backend socket

Let the user know if we are using a specific interface or source
address to contact the backend, as it is quite helpful to pinpoint
the issue.

Merge pull request #15043 from nixargh/docs_autoprimary_setup_add_warning

Docs: Add a warning about non-recursive queries during domain bootstrap on autosecondary

Merge pull request #15041 from miodvallat/guillotine

Exit gracefully on ^D on control-console

Merge pull request #15039 from miodvallat/augias_stables_take_1

Hair trimming in pdnsutil #1

dnsdist: Add a few missing settings to the YAML configuration

Update docs/modes-of-operation.rst

Co-authored-by: Peter van Dijk <peter.van.dijk@powerdns.com>

Merge pull request #15032 from metajiji/patch-1

dnsdist: Use getAddressInfo() instead python daemonized subprocess

dnsdist: Simplify the handle of flat settings from YAML

As suggested by Otto (thanks!)

dnsdist: Fix the handling of default values for YAML list of strings

Update docs/modes-of-operation.rst

Co-authored-by: Miod Vallat <miod.vallat@open-xchange.com>

Update docs/modes-of-operation.rst

Co-authored-by: Miod Vallat <miod.vallat@open-xchange.com>

return builder dir

dnsdist: Fix Rust warnings, be consistent in the naming of YAML objects

Merge remote-tracking branch 'origin/master' into docs_autoprimary_setup_add_warning

Docs: Add warning about non-recursive queries during automatic domain bootstrap

dnsdist: Use the better wording suggested by Otto in the upgrade notes

dnsdist: Yet more descriptions in the YAML documentation

dnsdist: More description in the YAML settings documentation

dnsdist: Fix formatting

dnsdist: Exit on YAML configuration error

dnsdist: Use '_' between YAML words, not '-'

To be consistent with what the recursor is already doing.

dnsdist: Stop cleaning dnsdist-configuration-yaml-items-generated.cc and lib.rs

dnsdist: Fix clang-tidy warnings

dnsdist: Support inline and "detached to a file" Lua code in YAML

dnsdist: Fix a small formatting issue

dnsdist: Reduce the complexity of the YAML loading function

dnsdist: Implement XSK and eBPF via YAML

dnsdist: Add more YAML example to the documentation

dnsdist: Remove unused Rust validation functions

dnsdist: Handle the default LB policy in YAML configuration

dnsdist: Handle DNSCrypt in the YAML configuration

dnsdist: Generate the YAML documentation

dnstap: Clean up the dnstap header

dnsdist: Add regression tests using the yaml configuration

dnsdist: Clean up the dnsdist protobuf header

dnsdist: Add a tool to generate the yaml configuration

dnsdist: Load the yaml configuration

dnsdist: Use the rules factory to unit test them

dnsdist: Add yaml configuration

dnsdist: Actually move selectors and actions to proper factories

dnsdist: Generate C++ factory and Lua bindings code from YAML definitions

dnsdist: Add an optional YAML configuration switch

dnsdist: Move the Lua bindings for single items to a separate file

QClass: Construct from a string

dnsdist: Fix error message for invalid TLS version

dnsdist: Move rules to dnsdist-rules-factory.hh

dnsdist: Move DNSRule declaration to dnsdist-rules.hh

dnsdist: Add a dynamic block rules group register

dnsdist: Move HTTP rules to dnsdist-rules.cc

dnsdist: Stop using Actions for regular processing

dnsdist: Stop using `LimitTTLResponseAction` to limit TTL

dnsdist: Move handling of backend parameters out of the Lua code

dnsdist: Properly handle built-in LB policies

dnsdist: Make SNMP configuration immutable

dnsdist: Fix automatic computation of the number of TCP workers

The previous method didn't account for TCP workers for backends.

dnsdist: Move functions from dnsdist-actions.hh to dnsdist-actions.cc

dnsdist: Add a yaml-based description of settings

dnsdist: Add yaml-based descriptions of actions and selectors

Merge pull request #15030 from rgacogne/ddist-remove-tlscontext

dnsdist: Remove `TLSContext`

Merge pull request #15037 from rgacogne/ddist-document-stat-node-ffi-children

dnsdist: Document `dnsdist_ffi_stat_node_get_children_*` return children+node stats

If the control console fd is closed, handle this as a "quit" command.

Fixes #1694

Const me if you can.

Appease clang-tidy