Clang-tidy fixes for http session

Author: seanharmer

src/KDNetwork/http_session.cpp

@@ -179,7 +179,7 @@ std::shared_ptr<Socket> HttpSession::getConnection(const std::string &host, uint
     return connection.socket;
 }
 
-void HttpSession::returnConnection(const std::string &host, uint16_t port, bool secure, std::shared_ptr<Socket> socket)
+void HttpSession::returnConnection(const std::string &host, uint16_t port, bool secure, const std::shared_ptr<Socket> &socket)
 {
     // Don't return invalid or disconnected sockets
     if (!socket || socket->state() != Socket::State::Connected) {

src/KDNetwork/http_session.h

@@ -180,7 +180,7 @@ class KDNETWORK_EXPORT HttpSession
      * @param secure Whether the connection uses SSL
      * @param socket The connection to return to the pool
      */
-    void returnConnection(const std::string &host, uint16_t port, bool secure, std::shared_ptr<Socket> socket);
+    void returnConnection(const std::string &host, uint16_t port, bool secure, const std::shared_ptr<Socket> &socket);
 
     /**
      * @brief Clean up idle connections