client/http: Internal code improvement

Tectu · Nov 25, 2024 · 985799e · 985799e
1 parent 969f7a6
commit 985799e
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/lib/malloy/client/controller.hpp b/lib/malloy/client/controller.hpp
@@ -289,11 +289,11 @@ namespace malloy::client
 
                 // Run
                 conn->run(
-                    std::to_string(req.port()).c_str(),
                     req,
                     std::move(prom),
                     std::forward<Callback>(cb),
-                    std::forward<Filter>(filter));
+                    std::forward<Filter>(filter)
+                );
             });
 
             return err_channel;

diff --git a/lib/malloy/client/http/connection.hpp b/lib/malloy/client/http/connection.hpp
@@ -43,7 +43,6 @@ namespace malloy::client::http
         // Start the asynchronous operation
         void
         run(
-            char const* port,
             malloy::http::request<ReqBody> req,
             std::promise<malloy::error_code> err_channel,
             callback_t&& cb,
@@ -58,7 +57,7 @@ namespace malloy::client::http
             // Look up the domain name
             m_resolver.async_resolve(
                 m_req.base()[malloy::http::field::host],
-                port,
+                std::to_string(m_req.port()),
                 boost::beast::bind_front_handler(
                     &connection::on_resolve,
                     derived().shared_from_this()