]> git.ipfire.org Git - thirdparty/kea.git/commitdiff
[#2651] release changes
authorAndrei Pavel <andrei@isc.org>
Fri, 25 Nov 2022 21:51:51 +0000 (23:51 +0200)
committerAndrei Pavel <andrei@isc.org>
Fri, 25 Nov 2022 22:17:08 +0000 (00:17 +0200)
47 files changed:
.gitlab/issue_templates/release_checklist.md
ChangeLog
doc/examples/agent/comments.json
doc/examples/agent/simple.json
doc/examples/kea4/ha-load-balancing-server1-mt-with-tls.json
doc/examples/kea4/ha-load-balancing-server2-mt.json
doc/examples/netconf/simple-dhcp4.json
doc/examples/netconf/simple-dhcp6.json
doc/examples/template-ha-mt-tls/info.md
doc/examples/template-ha-mt-tls/kea-ca-1.conf
doc/examples/template-ha-mt-tls/kea-ca-2.conf
doc/examples/template-ha-mt-tls/kea-dhcp4-1.conf
doc/examples/template-ha-mt-tls/kea-dhcp4-2.conf
doc/examples/template-power-user-home/kea-ca-1.conf
doc/examples/template-power-user-home/kea-ca-2.conf
doc/sphinx/arm/classify.rst
doc/sphinx/arm/config.rst
doc/sphinx/arm/dhcp6-srv.rst
doc/sphinx/arm/ext-netconf.rst
doc/sphinx/arm/hooks-flex-id.rst
doc/sphinx/arm/hooks.rst
doc/sphinx/arm/install.rst
platforms.rst
src/bin/agent/agent.dox
src/bin/dhcp6/dhcp6.dox
src/bin/netconf/netconf_messages.mes
src/hooks/dhcp/high_availability/ha_service.cc
src/hooks/dhcp/high_availability/tests/ha_impl_unittest.cc
src/lib/asiodns/io_fetch.cc
src/lib/d2srv/d2_config.h
src/lib/dhcp/option_definition.h
src/lib/dhcp/pkt.h
src/lib/dhcp/tests/protocol_util_unittest.cc
src/lib/dhcpsrv/cfgmgr.h
src/lib/dhcpsrv/client_class_def.h
src/lib/dhcpsrv/dhcpsrv_messages.mes
src/lib/dhcpsrv/memfile_lease_mgr.cc
src/lib/dhcpsrv/pool.cc
src/lib/dhcpsrv/tests/iterative_allocation_state_unittest.cc
src/lib/dhcpsrv/tests/memfile_lease_extended_info_unittest.cc
src/lib/log/message_initializer.h
src/lib/pgsql/pgsql_exchange.h
src/lib/tcp/tcp_connection.h
src/lib/tcp/tcp_messages.mes
src/lib/tcp/tests/tcp_test_listener.h
src/lib/util/buffer.h
src/lib/yang/translator.h

index a37cce4ab6821e11c25f39d7351c7d61d0f46240..c0ee46f88ba3cf72e18be05790c143ed41f68ff8 100644 (file)
@@ -39,17 +39,17 @@ For new stable releases or maintenance releases, please don't use `kea-dev` buil
 The following steps may involve changing files in the repository.
 
 1. [ ] Run [update-code-for-release.py](https://gitlab.isc.org/isc-private/qa-dhcp/-/blob/master/kea/build/update-code-for-release.py) <br>
-   Example command: `GITLAB_TOKEN='...' ./update-code-for-release.py 1.9.7 'Apr 28, 2021' ~/isc/repos/kea/` <br>
-   Help: `GITLAB_TOKEN="..." update-code-for-release.py --help`<br>
+   Example command: `GITLAB_TOKEN='...' ./update-code-for-release.py 1.9.7 ~/isc/repos/kea/` <br>
+   Help: `GITLAB_TOKEN="..." ./update-code-for-release.py --help`<br>
    This script makes the following changes and actions:
    1. run prepare_kea_release.sh that does:
-   1.1. add release entries in ChangeLogs
-   1.2. update Kea version in configure.ac
-   1.3. update copyright years in files that were changed in current year
-   1.4. sort message files
-   1.5. regenerate message files headers
+        1. add release entries in ChangeLogs
+        1. update Kea version in configure.ac
+        1. update copyright years in files that were changed in current year
+        1. sort message files
+        1. regenerate message files headers
    2. regenerate parsers using Bison from Docker<br>
-   using with --upload:
+   With `--upload`:
    3. create an issue in GitLab for release changes in kea repo
    4. create branches and merge requests for kea and kea-premium
    5. commit the changes in both repos
index eb2f5e24d5fb331c46067a9a5fd46f77a684e9a2..cef30030fcd53b27039fd45219ab83c437a9b0e7 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,5 @@
+Kea 2.3.3 (development) released on November 30, 2022
+
 2078.  [build]         andrei
        The library version numbers have been bumped for the Kea 2.3.3
        development release.
@@ -15,7 +17,7 @@
 2076.  [doc]           razvan
        Add documentation for template classes and the new parameter
        'template-test' which evaluates to a string value. This indicates
-       if the class is spaning classes with the name containing the
+       if the class is spawning classes with the name containing the
        evaluated value: 'SPAWN_<template-class-name>_<evaluated-value>'
        only if the value is not empty string. This will also add the
        packet to the <template-class-name> class.
index 6f40bdbb20af9ff51c48fcfce0187b85b086e338..10c82e3fa3795869d5f2cd7fafa540fef93e3a46 100644 (file)
@@ -41,7 +41,7 @@
         {
             "dhcp4":
             {
-                "comment": "control socket for DHCP4 server",
+                "comment": "control socket for DHCPv4 server",
                 "socket-type": "unix",
                 "socket-name": "/tmp/kea4-ctrl-socket"
             }
index 6c5af60e71c0b8c331a350fa05cb49762b6f7429..f48a5ecca4bbb180a51056aa493556a333c45aa8 100644 (file)
@@ -84,7 +84,7 @@
             // This is how the Agent can communicate with the DHCPv4 server.
             "dhcp4":
             {
-                "comment": "socket to DHCP4 server",
+                "comment": "socket to DHCPv4 server",
                 "socket-type": "unix",
                 "socket-name": "/tmp/kea4-ctrl-socket"
             },
index ff094a3387f50c9335c456c01182ef3386cc748d..e6f2f6d06672d6b1819b274ce71b046618a2d954 100644 (file)
                     // further. The default value is 60000ms (60 seconds).
                     "sync-timeout": 60000,
                     // To not experience performance degradation when the Kea server is
-                    // processing packets on multiple threads, the High Availablility module
+                    // processing packets on multiple threads, the High Availability module
                     // must have multi-threading enabled.
                     "multi-threading": {
                         "enable-multi-threading": true,
index 5f01cf4caf24eaf15c646339f1b622482f654fa7..51be49a82c39eabc9f6e0be19200b28c75868ff7 100644 (file)
                     // further. The default value is 60000ms (60 seconds).
                     "sync-timeout": 60000,
                     // To not experience performance degradation when the Kea server is
-                    // processing packets on multiple threads, the High Availablility module
+                    // processing packets on multiple threads, the High Availability module
                     // must have multi-threading enabled.
                     "multi-threading": {
                         "enable-multi-threading": true,
index 56407b2da91a8c8a7b401116c3a1c1fd6b5688c0..3e2105e2afc8858287f397f93daa1bacdfaceb47 100644 (file)
@@ -61,7 +61,7 @@
                 },
 
                 // Comment is optional. You can put some notes here.
-                "comment": "Kea DHCP4 server serving network on floor 13"
+                "comment": "Kea DHCPv4 server serving network on floor 13"
             }
 
         },
index ef07dd8f717a63199c45c86c5ca84c452e2a543e..4393f67606c323c6edabf2f5bf0cdf983c9136e5 100644 (file)
@@ -62,7 +62,7 @@
                 },
 
                 // Comment is optional. You can put some notes here.
-                "comment": "Kea DHCP6 server serving network on floor 13"
+                "comment": "Kea DHCPv6 server serving network on floor 13"
             }
 
         },
index 02e9f60e311c099da17acee6c2af1cab6c6d4f4f..d87f70c7c301caa4d427842333b1ae8a5eab4470 100644 (file)
@@ -55,7 +55,7 @@ The whole subnet is split into dynamic pools:
 To deploy this setup, follow the steps provided in the power user home setup with the following distinctions:
 
 1. Install CA only if the administrator is planning to manage Kea using RESTful API.
-   Otherwise, the High Availablility Kea server with multi-threading does not require CA to run.
+   Otherwise, the High Availability Kea server with multi-threading does not require CA to run.
 
 2. Alter the following to match the local setup:
 
index 127e7869243e268f25b84bd0b29aeee355257361..38aed10613060995f2021e519d60c90f0e20683c 100644 (file)
@@ -40,7 +40,7 @@
             // This is how the Agent can communicate with the DHCPv4 server.
             "dhcp4":
             {
-                "comment": "socket to DHCP4 server",
+                "comment": "socket to DHCPv4 server",
                 "socket-type": "unix",
                 "socket-name": "/tmp/kea4-ctrl-socket"
             },
index c7e42b02997308f4b388317fb7bc44a73dcbd373..1f08beaffee89728a8ad9ade4a5eeb99367509f1 100644 (file)
@@ -40,7 +40,7 @@
             // This is how the Agent can communicate with the DHCPv4 server.
             "dhcp4":
             {
-                "comment": "socket to DHCP4 server",
+                "comment": "socket to DHCPv4 server",
                 "socket-type": "unix",
                 "socket-name": "/tmp/kea4-ctrl-socket"
             },
index c586f803775bf7eab4e7efde4e5f1def659b9c06..e50d548322780a884d6f77dbd27bfd7cfce8e509 100644 (file)
                     "sync-timeout": 60000,
                     // Multi-threading parameters.
                     // To not experience performance degradation when the Kea server is
-                    // processing packets on multiple threads, the High Availablility module
+                    // processing packets on multiple threads, the High Availability module
                     // must have multi-threading enabled.
                     "multi-threading": {
                         "enable-multi-threading": true,
index 893bb9a7001c4d25aba663d720b672a025f8bc0e..2b17262f98d7994e9f35936a7023ea49fa0d7370 100644 (file)
                     "sync-timeout": 60000,
                     // Multi-threading parameters.
                     // To not experience performance degradation when the Kea server is
-                    // processing packets on multiple threads, the High Availablility module
+                    // processing packets on multiple threads, the High Availability module
                     // must have multi-threading enabled.
                     "multi-threading": {
                         "enable-multi-threading": true,
index d7749aee340b85972f50fa906be9f5b37334ed06..2620ea007ebbcb422f73a06ef3ab9c8dcd357120 100644 (file)
@@ -16,7 +16,7 @@
             // This is how the Agent can communicate with the DHCPv4 server.
             "dhcp4":
             {
-                "comment": "socket to DHCP4 server",
+                "comment": "socket to DHCPv4 server",
                 "socket-type": "unix",
                 "socket-name": "/tmp/kea4-ctrl-socket"
             },
index 59e4967bad7a8fc73da94bc968e26ce6a6782749..dccf5d9d87d640c0a5f4f1cb611a6fb65088cc02 100644 (file)
@@ -16,7 +16,7 @@
             // This is how the Agent can communicate with the DHCPv4 server.
             "dhcp4":
             {
-                "comment": "socket to DHCP4 server",
+                "comment": "socket to DHCPv4 server",
                 "socket-type": "unix",
                 "socket-name": "/tmp/kea4-ctrl-socket"
             },
index b2d9bb0ffebd8c0cea26ea7ac34b6f4ed3a0ae8b..3bb57c437120b091e260242961022f3011b75146 100644 (file)
@@ -790,7 +790,7 @@ ASCII, then the spawned class will use the ``SPAWN_Client-ID_foobar`` name
 effectively associating the regular class to the packet. In this second case,
 both ``SPAWN_Client-ID_foobar`` and ``Client-ID`` classes will be associated
 with the packet.
-The ``test`` expression on the regular classs ``SPAWN_Client-ID_foobar`` is not
+The ``test`` expression on the regular class ``SPAWN_Client-ID_foobar`` is not
 mandatory and can be omitted, but it is used here with a different match
 expression for example purposes.
 
@@ -836,7 +836,7 @@ is not mandatory that the flag be set to ``true``.
    options, are associated with the spawned class. This permits configuring
    limits which apply for all packets associated with a class spawned at
    runtime, according to the ``template-test`` expression in the parent template
-   class. For a more detaild description on how to configure limits using the
+   class. For a more detailed description on how to configure limits using the
    limits hooks library see the :ref:`hooks-limits-configuration`.
    For example, using the configuration below, ingress DHCPv6 packets that have
    client ID values (in the format expressed by the Kea evaluator) ``foobar``
index cb29dcc2b8d561d03662980996f3a9ef58cf17e1..c54380ee6d77ed22d7168b99a34eeb6517dbfcb0 100644 (file)
@@ -195,7 +195,7 @@ User context can be added and edited in structures supported by commands.
 We encourage Kea users to utilize these functions to store information
 used by other systems and custom hooks.
 
-For example, the `subnet4-update` command can be used to add user context data
+For example, the ``subnet4-update`` command can be used to add user context data
 to an existing subnet.
 
 ::
@@ -214,7 +214,7 @@ The same can be done with many other commands like lease6-add etc.
 Kea also uses user context to store non-standard data.
 Currently, only :ref:`dhcp4-store-extended-info` uses this feature.
 
-When enabled, it adds the ISC key in `user-context` to differentiate automatically
+When enabled, it adds the ISC key in ``user-context`` to differentiate automatically
 added content.
 
 Example of relay information stored in a lease:
index d07163ef9144f5730347331f20db0e8f62c52779..bda1c1ad182c44e8daa37f736286e8cfbad08c67 100644 (file)
@@ -2667,7 +2667,7 @@ servers set to 2001:db8:0::1 and 2001:db8:2::1.
 This example shows a configuration using an automatically generated
 ``VENDOR_CLASS_`` class. The administrator of the network has decided that
 addresses in the range 2001:db8:1::1 to 2001:db8:1::ffff are to be
-managed by the DHCP6 server and that only clients belonging to the
+managed by the DHCPv6 server and that only clients belonging to the
 eRouter1.0 client class are allowed to use that pool.
 
 ::
index 4234cd508fbaef6dc36d6fbbd8a8af959a03af5f..a1c9ce382205518ed68b6a7e5d42b64dddadf229 100644 (file)
@@ -564,7 +564,7 @@ Kea sources.
                // This is how kea-netconf can communicate with the DHCPv4 server.
                "dhcp4":
                {
-                   "comment": "DHCP4 server",
+                   "comment": "DHCPv4 server",
                    "model": "kea-dhcp4-server",
                    "control-socket":
                    {
index 664d9c17299a8e8cf77c0fa4881d85955e8c9bcd..2c11807d17797b2e13ac2cfbac45eb56029eddc8 100644 (file)
@@ -238,7 +238,7 @@ from incoming IPv6 packets. This parameter is ignored by the Kea DHCPv4 server.
 If the packet contains only one IA_NA, the IAID value will be changed to ``0``
 and stored as such in the lease storage. Similarly if the packet contains only
 one IA_PD, the IAID value will be changed to ``0`` and stored as such in the
-lease storage. The IAID is restored to its intitial value in the response back
+lease storage. The IAID is restored to its initial value in the response back
 to the client. The change is visible in the identifier expression if the IAID is
 part of the expression.
 
index 3b7ef06bd7f3eb913154c64c2119bebd92c38594..dad0be72d6f347c5fe83c6a81808faebb2fc3ce2 100644 (file)
@@ -22,7 +22,7 @@ to load those libraries.
 
 Hook libraries are loaded by individual Kea processes, not by Kea as a
 whole. This means, among other things, that it is possible to associate one set
-of libraries with the DHCP4 server and a different set with the DHCP6
+of libraries with the DHCPv4 server and a different set with the DHCPv6
 server.
 
 It is also possible for a process to load
index e360bd9cf63548129293fd50ecd15f34376ce31d..33b77b2ef74beb36795a193c32b7fe47e47febeb 100644 (file)
@@ -88,7 +88,7 @@ during upgrade depending on which packages were originally installed:
 
 If your goal is to have the entire Kea software suite installed, it is
 recommended that you simply ``apt install isc-kea`` after upgrading, which
-will install all of the relevent subpackages that make up Kea.
+will install all of the relevant subpackages that make up Kea.
 
 This upgrade path hiccup is not present on RPM and Alpine systems, however
 if you experience issues with upgrading past 2.3.1, please inform us on the
index cea1652fedd63039c357d55fd3c160ad90650821..6958dd810fd31ecd67a811d8e267b2ea39f25dfa 100644 (file)
@@ -20,10 +20,10 @@ Regularly Tested Platforms
 Kea is officially supported on Alpine, CentOS, Fedora, Ubuntu, Debian,
 and FreeBSD systems. Kea-|release| builds have been tested on:
 
-* Alpine â€” 3.13, 3.14, 3.15, 3.16
+* Alpine â€” 3.14, 3.15, 3.16
 * CentOS â€” 7
 * Debian â€” 10, 11
-* Fedora â€” 35, 36
+* Fedora â€” 36
 * FreeBSD â€” 12, 13
 * RHEL â€” 8, 9
 * Ubuntu â€” 18.04, 20.04, 22.04
@@ -38,29 +38,29 @@ ISC makes every effort to fix bugs on these platforms, but may be unable to
 do so quickly due to lack of hardware, less familiarity on the part of
 engineering staff, and other constraints.
 
-* FreeBSD â€” 11
-* macOS â€” 10.15, 11, 12
+* macOS â€” 11, 12, 13
 
 Community-Maintained
 --------------------
 
-These systems may not all have the required dependencies for building Kea
-easily available, although it is possible in many cases to compile
-those directly from source. The community and interested parties may wish
-to help with maintenance, and we welcome patch contributions, although we
-cannot guarantee that we will accept them. All contributions are
-assessed against the risk of adverse effect on officially supported
-platforms.
+These systems have once been regularly tested, but official support for it has
+been abandoned, usually due to discontinued support on their own part. Older
+versions may not have the required dependencies for building Kea easily
+available, although it is possible in many cases to compile on those directly
+from source. The community and interested parties may wish to help with
+maintenance, and we welcome patch contributions, although we cannot guarantee
+that we will accept them. All contributions are assessed against the risk of
+adverse effect on officially supported platforms.
 
 These include platforms past their respective EOL dates, such as:
 
-* Alpine â€” 3.10 (1 May 2021), 3.11, 3.12
-* CentOS â€” 6 (30 November 2020), 8 (31 December 2021)
-* Debian â€” 8 (30 June 2020), 9 (June 30, 2022)
-* Fedora â€” 31, 32, 33, 34
-* FreeBSD â€” 10 (31 October 2018)
-* macOS â€” 10.13, 10.14
-* Ubuntu â€” 14.04, 18.10, 19.04, 19.10, 21.04
+* Alpine â€” 3.10, 3.11, 3.12, 3.13 (EOL 01 November 2022)
+* CentOS â€” 6, 8 (EOL 31 December 2021)
+* Debian â€” 8, 9 (EOL 30 June 2022)
+* Fedora â€” 31, 32, 33, 34, 35 (EOL 13 December 2022)
+* FreeBSD â€” 10, 11 (EOL 30 September 2021)
+* macOS â€” 10.13, 10.14, 10.15 (EOL 12 September 2022)
+* Ubuntu â€” 14.04, 18.10, 19.04, 19.10, 21.04 (EOL 20 January 2022)
 
 Unsupported Platforms
 ---------------------
index 9e1f69d47158a49cf24f498b20347cbb6d8ac093..b8386777fbf291b9b23ab3b680047483674e0849 100644 (file)
@@ -1,4 +1,4 @@
-// Copyright (C) 2017 Internet Systems Consortium, Inc. ("ISC")
+// Copyright (C) 2017-2022 Internet Systems Consortium, Inc. ("ISC")
 //
 // This Source Code Form is subject to the terms of the Mozilla Public
 // License, v. 2.0. If a copy of the MPL was not distributed with this
@@ -122,7 +122,7 @@ It was eventually decided to not implement the secure layer within Kea for the f
   burden on Kea maintenance.
 
 In the installations where securing command channel is critical (most of the
-installations?), a reverse HTTP proxy can be setup using one of the third
+installations?), a reverse HTTP proxy can be set up using one of the third
 party HTTP server implementations, e.g. Apache, nginx etc.
 
 */
index 46c6a24af8b7a74cb46c76b632c52af4cfe1603f..661125349a4432a68d99c993e844f25371fd6354 100644 (file)
@@ -1,4 +1,4 @@
-// Copyright (C) 2012-2018 Internet Systems Consortium, Inc. ("ISC")
+// Copyright (C) 2012-2022 Internet Systems Consortium, Inc. ("ISC")
 //
 // This Source Code Form is subject to the terms of the Mozilla Public
 // License, v. 2.0. If a copy of the MPL was not distributed with this
index 18b9a58e515bdc5aa9ccd5b1ed1f359e41b6318c..0516385a0e3ea63fdd6541ae2110bdad6d1fcd60 100644 (file)
@@ -75,6 +75,12 @@ This warning message indicates that a module used by Kea is not at the
 right revision in the sysrepo repository. The name, expected and
 available revisions of the module are printed.
 
+% NETCONF_NOTIFICATION_RECEIVED Received notification of type %1 for module %1: %2
+This informational message logs any YANG notification that has been signaled
+by the server, sent to kea-netconf which then was forwarded to subscribed
+clients. To achieve this, kea-netconf subscribes itself as a client to all
+notifications for the configured module.
+
 % NETCONF_NOT_SUBSCRIBED_TO_NOTIFICATIONS subscribing to notifications for %1 server with %2 module failed: %3
 The warning message indicates that kea-netconf got an error subscribing to
 notifications for a Kea server. The most probable cause is probably that the
@@ -82,12 +88,6 @@ model that kea-netconf subscribed to does not have any notification nodes, but
 there may be other more unxpected causes as well.
 The server name, module name and the error are printed.
 
-% NETCONF_NOTIFICATION_RECEIVED Received notification of type %1 for module %1: %2
-This informational message logs any YANG notification that has been signaled
-by the server, sent to kea-netconf which then was forwarded to subscribed
-clients. To achieve this, kea-netconf subscribes itself as a client to all
-notifications for the configured module.
-
 % NETCONF_RUN_EXIT application is exiting the event loop
 This is a debug message issued when kea-netconf exits its
 event loop. This is a normal step during kea-netconf shutdown.
index f23b60339386078f5376240cf9fb190b44010bd8..a3951dc7cb40d6c0d515b6a158e915aecd3fdd33 100644 (file)
@@ -2996,7 +2996,7 @@ HAService::verifyAsyncResponse(const HttpResponsePtr& response, int& rcode) {
     // The empty status can occur for the lease6-bulk-apply command. In that
     // case, the response may contain conflicted or erred leases within the
     // arguments, rather than globally. For other error cases let's construct
-    // the error message from tyhe global values.
+    // the error message from the global values.
     if (rcode != CONTROL_RESULT_EMPTY) {
         // Include an error text if available.
         if (args && args->getType() == Element::string) {
index 38c1d9a02a2a008516e27205e4410920ff9013d3..514128183a7948b730f486172ced4ba075d36b94 100644 (file)
@@ -1,4 +1,4 @@
-// Copyright (C) 2018-2021 Internet Systems Consortium, Inc. ("ISC")
+// Copyright (C) 2018-2022 Internet Systems Consortium, Inc. ("ISC")
 //
 // This Source Code Form is subject to the terms of the Mozilla Public
 // License, v. 2.0. If a copy of the MPL was not distributed with this
index 2e29b47ab0494b55d9c1e100ac4110698a29b866..980a6009b65a23719a46dde3cfd80b136f29940c 100644 (file)
@@ -1,4 +1,4 @@
-// Copyright (C) 2011-2021 Internet Systems Consortium, Inc. ("ISC")
+// Copyright (C) 2011-2022 Internet Systems Consortium, Inc. ("ISC")
 //
 // This Source Code Form is subject to the terms of the Mozilla Public
 // License, v. 2.0. If a copy of the MPL was not distributed with this
@@ -247,8 +247,8 @@ IOFetch::operator()(boost::system::error_code ec, size_t length) {
             }
         }
 
-        // If we timeout, we stop, which will can cancel outstanding I/Os and
-        // shutdown everything.
+        // If we timeout, we stop, which cancels outstanding I/O operations and
+        // shutdown everything.
         if (data_->timeout != -1) {
             data_->timer.expires_from_now(boost::posix_time::milliseconds(
                 data_->timeout));
index 6162ec5ea15154ed008af09adc9ad2af9e1b0772..cb141ad6b77f6573f40dc01d87b620df6fc2ad53 100644 (file)
@@ -1,4 +1,4 @@
-// Copyright (C) 2013-2021 Internet Systems Consortium, Inc. ("ISC")
+// Copyright (C) 2013-2022 Internet Systems Consortium, Inc. ("ISC")
 //
 // This Source Code Form is subject to the terms of the Mozilla Public
 // License, v. 2.0. If a copy of the MPL was not distributed with this
@@ -266,14 +266,14 @@ typedef boost::shared_ptr<D2Params> D2ParamsPtr;
 class TSIGKeyInfo : public isc::data::UserContext, public isc::data::CfgToElement {
 public:
     /// @brief Defines string values for the supported TSIG algorithms
-    //@{
+    //@{
     static const char* HMAC_MD5_STR;
     static const char* HMAC_SHA1_STR;
     static const char* HMAC_SHA256_STR;
     static const char* HMAC_SHA224_STR;
     static const char* HMAC_SHA384_STR;
     static const char* HMAC_SHA512_STR;
-    //}@
+    /// @}
 
     /// @brief Constructor
     ///
index a9316acf2ce8f3b11484742214eea3c8ced510bc..08a5126313788d8b49865796563f6eaa34b6394a 100644 (file)
@@ -1,4 +1,4 @@
-// Copyright (C) 2012-2021 Internet Systems Consortium, Inc. ("ISC")
+// Copyright (C) 2012-2022 Internet Systems Consortium, Inc. ("ISC")
 //
 // This Source Code Form is subject to the terms of the Mozilla Public
 // License, v. 2.0. If a copy of the MPL was not distributed with this
@@ -345,7 +345,7 @@ public:
     /// @brief Return array type indicator.
     ///
     /// The method returns the bool value to indicate whether the option is a
-    /// single value or an array of values.
+    /// single value or an array of values.
     ///
     /// @return true if option comprises an array of values.
     bool getArrayType() const { return (array_type_); }
index 7c8c3025068741b758498ce82bd3310b9e7ee4f4..77e9a2b0f1b8e2526788ae4c65fb11a4955372b1 100644 (file)
@@ -379,7 +379,7 @@ public:
     /// Kea configuration. To prevent this, option copying should be
     /// enabled prior to passing the pointer to a packet to a hook library.
     ///
-    /// Note that only only does this method causes the server to copy
+    /// Not only does this method cause the server to copy
     /// an option, but the copied option also replaces the original
     /// option within the packet. The option can be then freely modified
     /// and the modifications will only affect the instance of this
index 45c2fd5d6d3e273c84b86612bb101967d7e59452..c89bf0f9324d573e9fa6a2b66378677f1888ea39 100644 (file)
@@ -278,7 +278,7 @@ TEST(ProtocolUtilTest, writeEthernetHeaderBroadcast) {
     HWAddrPtr remote_hw_addr(new HWAddr(dest_hw_addr, 6, 1));
     ASSERT_NO_THROW(pkt->setRemoteHWAddr(remote_hw_addr));
 
-    // Set the broadcast flasg.
+    // Set the broadcast flags.
     pkt->setFlags(pkt->getFlags() | Pkt4::FLAG_BROADCAST_MASK);
 
     // Construct the ethernet header using HW addresses stored
@@ -331,7 +331,7 @@ TEST(ProtocolUtilTest, writeEthernetHeaderBroadcastRelayed) {
     HWAddrPtr remote_hw_addr(new HWAddr(dest_hw_addr, 6, 1));
     ASSERT_NO_THROW(pkt->setRemoteHWAddr(remote_hw_addr));
 
-    // Set the broadcast flasg.
+    // Set the broadcast flags.
     pkt->setFlags(pkt->getFlags() | Pkt4::FLAG_BROADCAST_MASK);
 
     // Set a gateway address: the broadcast flag is now for
index 0302cf54e19892b85fa2b53ce986484bb2b2e4f3..d34b1c5e87eb7b8cbf7b2d9c25c4f05630a5257c 100644 (file)
@@ -1,4 +1,4 @@
-// Copyright (C) 2012-2019 Internet Systems Consortium, Inc. ("ISC")
+// Copyright (C) 2012-2022 Internet Systems Consortium, Inc. ("ISC")
 //
 // This Source Code Form is subject to the terms of the Mozilla Public
 // License, v. 2.0. If a copy of the MPL was not distributed with this
@@ -160,7 +160,7 @@ public:
     /// @brief Removes staging configuration.
     ///
     /// This function should be called when there is a staging configuration
-    /// (likely created in the previous configuration attempt) but the entirely
+    /// (likely created in the previous configuration attempt) but the entire
     /// new configuration should be created. It removes the existing staging
     /// configuration and the next call to @c CfgMgr::getStagingCfg will return a
     /// fresh (default) configuration.
index ca7f013d2c764b2340788de0ab85d65f066223b0..4e8e981314ce49ef9ffaf8f6a0efc9b7666d2895 100644 (file)
@@ -318,7 +318,7 @@ public:
     /// If incoming packet is associated with the template class, the name of
     /// generated spawned class is prepended with this prefix.
     /// For example, a packet that associates with the template class "FOO" by
-    /// evaluating the templeta class expression to BAR will cause the packet to
+    /// evaluating the template class expression to BAR will cause the packet to
     /// be assigned to class SPAWN_FOO_BAR.
     static const std::string SPAWN_CLASS_PREFIX;
 };
index f1e2ce991797bf659d61e8182439e689bd0fc531..d2105472d9fbbbd47272dd291cf2ef7e9dcc94bd 100644 (file)
@@ -395,14 +395,14 @@ and the fact tables are enabled or disabled are displayed.
 The code has issued a begin transaction call.  For the memory file database, this is
 a no-op.
 
-% DHCPSRV_MEMFILE_BUILD_EXTENDED_INFO_TABLES6_ERROR building extended info tables got an exception on the lease for %1: %2
-A debug message issued when the server is building extended info tables and
-receives an exception processing a lease.
-
 % DHCPSRV_MEMFILE_BUILD_EXTENDED_INFO_TABLES6 building extended info tables saw %1 leases, extended info sanity checks modified %2 / updated %3 leases and %4 leases were entered into tables
 Extended info tables build was finished. Some statistics are displayed, the
 updated in database is returned to the command interface.
 
+% DHCPSRV_MEMFILE_BUILD_EXTENDED_INFO_TABLES6_ERROR building extended info tables got an exception on the lease for %1: %2
+A debug message issued when the server is building extended info tables and
+receives an exception processing a lease.
+
 % DHCPSRV_MEMFILE_COMMIT committing to memory file database
 The code has issued a commit call.  For the memory file database, this is
 a no-op.
index a36bc056ecd7b7e3fbbdca707e00114522270cb3..1e5677e46bed708c81150f7b728d23f880b80aa6 100644 (file)
@@ -2444,8 +2444,9 @@ Memfile_LeaseMgr::getLeaseLimit(ConstElementPtr parent, Lease::Type ltype, size_
 }
 
 namespace {
+
 std::string
-idtoText(const OptionBuffer& id) {
+idToText(const OptionBuffer& id) {
     std::stringstream tmp;
     tmp << std::hex;
     bool delim = false;
@@ -2460,6 +2461,7 @@ idtoText(const OptionBuffer& id) {
     }
     return (tmp.str());
 }
+
 } // anonymous namespace
 
 Lease4Collection
@@ -2479,7 +2481,7 @@ Memfile_LeaseMgr::getLeases4ByRelayId(const OptionBuffer& relay_id,
               DHCPSRV_MEMFILE_GET_RELAYID4)
         .arg(page_size.page_size_)
         .arg(lower_bound_address.toText())
-        .arg(idtoText(relay_id))
+        .arg(idToText(relay_id))
         .arg(qry_start_time)
         .arg(qry_end_time);
 
@@ -2525,7 +2527,7 @@ Memfile_LeaseMgr::getLeases4ByRemoteId(const OptionBuffer& remote_id,
               DHCPSRV_MEMFILE_GET_REMOTEID4)
         .arg(page_size.page_size_)
         .arg(lower_bound_address.toText())
-        .arg(idtoText(remote_id))
+        .arg(idToText(remote_id))
         .arg(qry_start_time)
         .arg(qry_end_time);
 
@@ -2690,7 +2692,7 @@ Memfile_LeaseMgr::getLeases6ByRemoteId(const OptionBuffer& remote_id,
               DHCPSRV_MEMFILE_GET_REMOTEID6)
         .arg(page_size.page_size_)
         .arg(lower_bound_address.toText())
-        .arg(idtoText(remote_id))
+        .arg(idToText(remote_id))
         .arg(link_addr.toText())
         .arg(static_cast<unsigned>(link_len));
 
index 04788b8b520f27545976ccd0930660282494dd70..f5a88d04305fb2c0291b2d9ca50a28156cb58217 100644 (file)
@@ -1,4 +1,4 @@
-// Copyright (C) 2012-2020 Internet Systems Consortium, Inc. ("ISC")
+// Copyright (C) 2012-2022 Internet Systems Consortium, Inc. ("ISC")
 //
 // This Source Code Form is subject to the terms of the Mozilla Public
 // License, v. 2.0. If a copy of the MPL was not distributed with this
index 6de62eddb0409fcdda4517b4773099bdeee1b084..8b0bd243564d64e11c23097ecc86c6647ee71bb8 100644 (file)
@@ -171,4 +171,4 @@ TEST(IterativeAllocationStateTest, poolLastAllocated6) {
     EXPECT_FALSE(state->isLastAllocatedValid());
 }
 
-} // end of anonymouns namespace
+}  // anonymous namespace
index e12fb91ce2991c5b57628665ce23a1a263fdee24..f9e26d2f6134d2660ecc95c7876cd21cab88d7fc 100644 (file)
@@ -1241,7 +1241,7 @@ TEST_F(MemfileExtendedInfoTest, updateLease6ignore) {
 
     // Note that with persist when the database is reloaded the user context
     // is still there so tables will be updated: the ACTION_IGNORE setting
-    // has no persistent effect: instead clenup the user context before
+    // has no persistent effect: instead cleanup the user context before
     // calling updateLease6...
 }
 
index 491a578ed09deefd4f521ac47a020df54e9f31f7..7f40f9b0ee2f6c1600f8c666d8a561ba8b85c1c3 100644 (file)
@@ -1,4 +1,4 @@
-// Copyright (C) 2011-2015,2017 Internet Systems Consortium, Inc. ("ISC")
+// Copyright (C) 2011-2022 Internet Systems Consortium, Inc. ("ISC")
 //
 // This Source Code Form is subject to the terms of the Mozilla Public
 // License, v. 2.0. If a copy of the MPL was not distributed with this
@@ -38,7 +38,7 @@ typedef boost::shared_ptr<LoggerDuplicatesList> LoggerDuplicatesListPtr;
 /// This is a helper class to add a set of message IDs and associated text to
 /// the global dictionary.
 ///
-/// It should be declared outside an execution unit and initialized with a
+/// It should be declared outside an execution unit and initialized with
 /// an array of values, alternating identifier, associated text and ending with
 /// a NULL, e.g.
 ///
index c6172c10a9e226e8814f24b30a57b436788409a6..22ec7afe0ce278c9f6c10d5af131c090adb314b0 100644 (file)
@@ -764,7 +764,6 @@ public:
     /// will be stored.
     /// @param buffer_size size of the output buffer
     /// @param[out] bytes_converted number of bytes converted
-    /// value
     ///
     /// @throw  DbOperationError if the value cannot be fetched or is
     /// invalid.
index f64abf740ece18aa70b9022e6b4e9d3dca3a7ef6..bf37bb01144f75725e8f322c08d030f99bff904b 100644 (file)
@@ -307,7 +307,7 @@ public:
 
     /// @brief Returns an empty end point.
     ///
-    /// @return an unitialized endpoint.
+    /// @return an uninitialized endpoint.
     static const boost::asio::ip::tcp::endpoint& NO_ENDPOINT() {
         static boost::asio::ip::tcp::endpoint endpoint;
         return (endpoint);
index 158244d9d99998722a1acee2437f7712fdbbb53c..7c97369c7a91bd95a8a45bb2d93c827430838d64 100644 (file)
@@ -6,6 +6,20 @@
 
 $NAMESPACE isc::tcp
 
+% MT_TCP_LISTENER_MGR_STARTED MtTcpListenerMgr started with %1 threads, listening on %2:%3, use TLS: %4
+This debug messages is issued when an MtTcpListenerMgr has been started to
+accept connections.  Arguments detail the number of threads that the listener
+is using, the address and port at which it is listening, and if TLS is used
+or not.
+
+% MT_TCP_LISTENER_MGR_STOPPED MtTcpListenerMgr for %1:%2 stopped.
+This debug messages is issued when the MtTcpListenerMgr, listening
+at the given address and port, has completed shutdown.
+
+% MT_TCP_LISTENER_MGR_STOPPING Stopping MtTcpListenerMgr for %1:%2
+This debug messages is issued when the MtTcpListenerMgr, listening
+at the given address and port, has begun to shutdown.
+
 % TCP_CLIENT_REQUEST_RECEIVED received TCP request from %1
 This debug message is issued when the server finished receiving a TCP
 request from the remote endpoint. The address of the remote endpoint is
@@ -26,6 +40,10 @@ with the remote endpoint. The first argument specifies the address
 of the remote endpoint. The second argument specifies request timeout in
 seconds.
 
+% TCP_CONNECTION_REJECTED_BY_FILTER connection from %1 has been denied by the connection filter.
+This debug message is issued when the server's connection filter rejects
+a new connection based on the client's ip address.
+
 % TCP_CONNECTION_SHUTDOWN shutting down TCP connection from %1
 This debug message is issued when one of the TCP connections is shut down.
 The connection can be stopped as a result of an error or after the
@@ -61,9 +79,11 @@ transaction. This is proven to occur when the system clock is moved manually
 or as a result of synchronization with a time server. Any ongoing transactions
 will be interrupted. New transactions should be conducted normally.
 
-% TCP_CONNECTION_REJECTED_BY_FILTER connection from %1 has been denied by the connection filter.
-This debug message is issued when the server's connection filter rejects
-a new connection based on the client's ip address.
+% TCP_REQUEST_RECEIVED_FAILED An unexpected error occurred processing a request from %1, error: %2
+This error message is issued when an unexpected error occurred while the
+server attempted to process a received request. The first argument specifies
+the address of the remote endpoint. The second argument describes the nature
+error.
 
 % TCP_REQUEST_RECEIVE_START start receiving request from %1 with timeout %2
 This debug message is issued when the server starts receiving new request
@@ -71,12 +91,6 @@ over the established connection. The first argument specifies the address
 of the remote endpoint. The second argument specifies request timeout in
 seconds.
 
-% TCP_REQUEST_RECEIVED_FAILED An unexpected error occurred processing a request from %1, error: %2
-This error message is issued when an unexpected error occurred while the
-server attempted to process a received request. The first argument specifies
-the address of the remote endpoint. The second argument describes the nature
-error.
-
 % TCP_SERVER_RESPONSE_SEND sending TCP response %1 to %2
 This debug message is issued when the server is starting to send a TCP
 response to a remote endpoint. The first argument holds basic information
@@ -90,17 +104,3 @@ first argument specifies an address of the remote endpoint to which the
 response is being sent. The second argument provides a response in the
 textual form. The response is truncated by the logger if it is too large
 to be printed.
-
-% MT_TCP_LISTENER_MGR_STARTED MtTcpListenerMgr started with %1 threads, listening on %2:%3, use TLS: %4
-This debug messages is issued when an MtTcpListenerMgr has been started to
-accept connections.  Arguments detail the number of threads that the listener
-is using, the address and port at which it is listening, and if TLS is used
-or not.
-
-% MT_TCP_LISTENER_MGR_STOPPED MtTcpListenerMgr for %1:%2 stopped.
-This debug messages is issued when the MtTcpListenerMgr, listening
-at the given address and port, has completed shutdown.
-
-% MT_TCP_LISTENER_MGR_STOPPING Stopping MtTcpListenerMgr for %1:%2
-This debug messages is issued when the MtTcpListenerMgr, listening
-at the given address and port, has begun to shutdown.
index a09db5f3faaa70b1f6e7bbe9afcbd2ab91404ae3..5522484a84832dd71f3be592120db3caaa4ffb1d 100644 (file)
@@ -290,7 +290,7 @@ protected:
     /// @brief Id to use for the next connection.
     size_t next_connection_id_;
 
-    /// @brief Callback connection invokes in requestReceived to construct a respsonse.
+    /// @brief Callback connection invokes in requestReceived to construct a response.
     TcpTestConnection::ResponseHandler response_handler_;
 
 public:
index 3e569b299455872b17aa765d385e460e44013cc9..4f918566af10410e07c452f946247c8add5472f3 100644 (file)
@@ -282,7 +282,7 @@ protected:
 ///
 /// Note to developers: it may make more sense to introduce an abstract base
 /// class for the \c OutputBuffer and define the simple implementation as a
-/// concrete derived class.  That way we can provide flexibility for future
+/// concrete derived class.  That way we can provide flexibility for future
 /// extension such as more efficient buffer implementation or allowing users
 /// to have their own customized version without modifying the source code.
 /// We in fact considered that option, but at the moment chose the simpler
index 2cb38a33eb338be42b652eb3088fa347ae375d8e..5413824b40c0dfed20f016bee7997032ac68ca42 100644 (file)
@@ -202,9 +202,8 @@ public:
     /// in the sysrepo datastore by calling Session::getData(). It should be
     /// used sparingly in production code. It is primarily meant for unit tests.
     ///
-    /// @param xpath the xpath to be travelled
-    /// @param f the function to be called on the node itself and each
-    /// descendant
+    /// @param xpath the xpath of the root node belonging to the the tree being traversed
+    /// @param f the function to be called on the node itself and each descendant
     template <typename functor_t>
     void forAll(std::string const& xpath, functor_t f) const {
         std::optional<libyang::DataNode> const& data_node(session_.getData(xpath));