[#3552] Addressed comments

diff --git a/doc/sphinx/arm/security.rst b/doc/sphinx/arm/security.rst
index d729455fd70f9c09a10cd9676367fcbb035fd3e4..8ceb4f12ecd5e4a3c1cc892a2ae3d5a8b1e72f31 100644 (file)
--- a/doc/sphinx/arm/security.rst
+++ b/doc/sphinx/arm/security.rst
@@ -76,7 +76,7 @@ that must be used:
 
   Without these header files, or with a Botan version prior
   to 2.14.0, Kea cannot build as the TLS/HTTPS support is considered
-  as essential for security.
+  essential for security.
 
 - Very old Boost versions provide SSL support (based on OpenSSL)
   without offering a choice of the TLS version; Kea can still use them,

diff --git a/src/lib/asiolink/botan_tls.cc b/src/lib/asiolink/botan_tls.cc
index b95485ccc3d981014f07dbaacdb8eeae5adbf62e..0ec0ed5fdb059c1ada65e91a7253d58a0122cd22 100644 (file)
--- a/src/lib/asiolink/botan_tls.cc
+++ b/src/lib/asiolink/botan_tls.cc
@@ -182,7 +182,6 @@ public:
     static const std::vector<std::string> AllowedSignatureMethodsECDSA;
 };
 
-
 // Kea session manager.
 using KeaSessionManager = Botan::TLS::Session_Manager_Noop;
 

diff --git a/src/lib/asiolink/botan_tls.h b/src/lib/asiolink/botan_tls.h
index 6004672336c5d89ff65f303293e2fee2d534166b..8dc9e40b37d1508024e4253df894a3a61ffe0b5e 100644 (file)
--- a/src/lib/asiolink/botan_tls.h
+++ b/src/lib/asiolink/botan_tls.h
@@ -43,7 +43,7 @@ public:
 
     /// @brief Destructor.
     ///
-    /// @note The destructor can't be defined here because  a unique
+    /// @note The destructor can't be defined here because a unique
     /// pointer to an incomplete type is used.
     virtual ~TlsContext();
 

diff --git a/src/lib/cryptolink/botan_hmac.cc b/src/lib/cryptolink/botan_hmac.cc
index 14cb00f7e2eaf1ad4e5d12e29c9437614b15fbc2..fc4f773a80fb35a66d0834c55b75162462dc05dc 100644 (file)
--- a/src/lib/cryptolink/botan_hmac.cc
+++ b/src/lib/cryptolink/botan_hmac.cc
@@ -66,8 +66,6 @@ public:
             isc_throw(LibraryError, "Botan error: " << exc.what());
         }
 
-        // If the key length is larger than the block size, we hash the
-        // key itself first.
         try {
             // Botan 1.8 considers len 0 a bad key. 1.9 does not,
             // but we won't accept it anyway, and fail early

diff --git a/src/lib/http/tests/tls_client_unittests.cc b/src/lib/http/tests/tls_client_unittests.cc
index b0e0d659e6ebe5a6a7db9843f876f32f86dc1b1a..b19ebe07242445170bf27f8d47bed6ec8280ed52 100644 (file)
--- a/src/lib/http/tests/tls_client_unittests.cc
+++ b/src/lib/http/tests/tls_client_unittests.cc
@@ -37,7 +37,8 @@
 #include <http/tests/tls_response_creator_test.h>
 
 #ifdef WITH_BOTAN
-#define DISABLE_SOME_TESTS
+// All tests work with last Botan versions so commenting this.
+// #define DISABLE_SOME_TESTS
 #endif
 #ifdef WITH_OPENSSL
 #if !defined(LIBRESSL_VERSION_NUMBER) && (OPENSSL_VERSION_NUMBER < 0x10100000L)