[CODE HEALTH] Fix remaining misc-override-with-different-visibility warnings (#4280)

Author: thc1006

.github/workflows/clang-tidy.yaml

@@ -17,9 +17,9 @@ jobs:
       matrix:
         include:
           - cmake_options: all-options-abiv1-preview
-            warning_limit: 156
+            warning_limit: 151
           - cmake_options: all-options-abiv2-preview
-            warning_limit: 166
+            warning_limit: 161
     env:
       CC: /usr/bin/clang-22
       CXX: /usr/bin/clang++-22

CHANGELOG.md

@@ -44,6 +44,9 @@ Increment the:
         that do not receive explicit SpanLimits
   [#4258](https://github.com/open-telemetry/opentelemetry-cpp/pull/4258)
 
+* [CODE HEALTH] Fix remaining misc-override-with-different-visibility warnings
+  [#4280](https://github.com/open-telemetry/opentelemetry-cpp/pull/4280)
+
 Breaking changes:
 
 * [METRICS SDK] Rename Base2 Exponential Histogram Aggregation config field

ext/include/opentelemetry/ext/http/server/http_server.h

@@ -244,7 +244,7 @@ class HttpServer : private SocketTools::Reactor::SocketCallback
 
   void stop() { m_reactor.stop(); }
 
-protected:
+private:
   void onSocketAcceptable(SocketTools::Socket socket) override
   {
     LOG_TRACE("HttpServer: accepting socket fd=0x%llx", socket.m_sock);
@@ -331,6 +331,7 @@ class HttpServer : private SocketTools::Reactor::SocketCallback
     handleConnectionClosed(conn);
   }
 
+protected:
   bool sendMore(Connection &conn)
   {
     if (conn.sendBuffer.empty())
@@ -356,7 +357,6 @@ class HttpServer : private SocketTools::Reactor::SocketCallback
     return false;
   }
 
-protected:
   void handleConnectionClosed(Connection &conn)
   {
     LOG_TRACE("HttpServer: [%s] closed", conn.request.client.c_str());

ext/include/opentelemetry/ext/http/server/socket_tools.h

@@ -717,6 +717,7 @@ struct Reactor : protected common::Thread
     m_sockets.clear();
   }
 
+protected:
   /// <summary>
   /// Thread Loop for async events processing
   /// </summary>