connection_pool: extract effective_port into its own header

Author: ashtum

src/detail/connection_pool.cpp

@@ -11,6 +11,7 @@
 
 #include <boost/burl/error.hpp>
 
+#include "effective_port.hpp"
 #include "http_tunnel.hpp"
 #include "socks5_tunnel.hpp"
 
@@ -39,24 +40,6 @@ namespace detail
 namespace
 {
 
-std::string_view
-effective_port(const urls::url_view& url)
-{
-    if(url.has_port())
-        return url.port();
-
-    if(url.scheme() == "https")
-        return "443";
-
-    if(url.scheme() == "http")
-        return "80";
-
-    if(url.scheme() == "socks5" || url.scheme() == "socks5h")
-        return "1080";
-
-    return {};
-}
-
 std::string
 origin(urls::url_view url)
 {

src/detail/effective_port.cpp

@@ -0,0 +1,39 @@
+//
+// Copyright (c) 2026 Mohammad Nejati
+//
+// Distributed under the Boost Software License, Version 1.0. (See accompanying
+// file LICENSE_1_0.txt or copy at http://www.boost.org/LICENSE_1_0.txt)
+//
+// Official repository: https://github.com/cppalliance/burl
+//
+
+#include "effective_port.hpp"
+
+namespace boost
+{
+namespace burl
+{
+namespace detail
+{
+
+std::string_view
+effective_port(const urls::url_view& url) noexcept
+{
+    if(url.has_port())
+        return url.port();
+
+    if(url.scheme() == "https")
+        return "443";
+
+    if(url.scheme() == "http")
+        return "80";
+
+    if(url.scheme() == "socks5" || url.scheme() == "socks5h")
+        return "1080";
+
+    return {};
+}
+
+} // namespace detail
+} // namespace burl
+} // namespace boost

src/detail/effective_port.hpp

@@ -0,0 +1,31 @@
+//
+// Copyright (c) 2026 Mohammad Nejati
+//
+// Distributed under the Boost Software License, Version 1.0. (See accompanying
+// file LICENSE_1_0.txt or copy at http://www.boost.org/LICENSE_1_0.txt)
+//
+// Official repository: https://github.com/cppalliance/burl
+//
+
+#ifndef BOOST_BURL_SRC_DETAIL_EFFECTIVE_PORT_HPP
+#define BOOST_BURL_SRC_DETAIL_EFFECTIVE_PORT_HPP
+
+#include <boost/url/url_view.hpp>
+
+#include <string_view>
+
+namespace boost
+{
+namespace burl
+{
+namespace detail
+{
+
+std::string_view
+effective_port(const urls::url_view& url) noexcept;
+
+} // namespace detail
+} // namespace burl
+} // namespace boost
+
+#endif

test/unit/detail/effective_port.cpp

@@ -0,0 +1,62 @@
+//
+// Copyright (c) 2026 Mohammad Nejati
+//
+// Distributed under the Boost Software License, Version 1.0. (See accompanying
+// file LICENSE_1_0.txt or copy at http://www.boost.org/LICENSE_1_0.txt)
+//
+// Official repository: https://github.com/cppalliance/burl
+//
+
+// Test that header file is self-contained.
+#include "src/detail/effective_port.hpp"
+
+#include "test_suite.hpp"
+
+namespace boost
+{
+namespace burl
+{
+namespace detail
+{
+
+class effective_port_test
+{
+public:
+    void
+    testExplicitPort()
+    {
+        BOOST_TEST_EQ(effective_port("http://example.com:8080"), "8080");
+        BOOST_TEST_EQ(effective_port("ftp://example.com:21"), "21");
+        BOOST_TEST_EQ(effective_port("http://example.com:80"), "80");
+    }
+
+    void
+    testDefaultPort()
+    {
+        BOOST_TEST_EQ(effective_port("http://example.com"), "80");
+        BOOST_TEST_EQ(effective_port("https://example.com"), "443");
+        BOOST_TEST_EQ(effective_port("socks5://example.com"), "1080");
+        BOOST_TEST_EQ(effective_port("socks5h://example.com"), "1080");
+    }
+
+    void
+    testUnknownScheme()
+    {
+        BOOST_TEST_EQ(effective_port("ftp://example.com"), "");
+        BOOST_TEST_EQ(effective_port("ws://example.com"), "");
+    }
+
+    void
+    run()
+    {
+        testExplicitPort();
+        testDefaultPort();
+        testUnknownScheme();
+    }
+};
+
+TEST_SUITE(effective_port_test, "boost.burl.detail.effective_port");
+
+} // namespace detail
+} // namespace burl
+} // namespace boost