fix(win): avoid std::min (winsock min macro) in DNS timeout; bump 0.2.8

0.2.7 broke the Windows build: <winsock2.h> defines a min macro that mangles
std::min in the compiled-but-discarded Windows branch of connect()'s if
constexpr. Use a plain ternary.

Author: Sunrisepeak

mcpp.toml

@@ -1,7 +1,7 @@
 [package]
 namespace   = "mcpplibs"
 name        = "tinyhttps"
-version     = "0.2.7"
+version     = "0.2.8"
 description = "Minimal C++23 HTTP/HTTPS client with SSE streaming support"
 license     = "Apache-2.0"
 repo        = "https://github.com/mcpplibs/tinyhttps"

src/socket.cppm

@@ -98,9 +98,12 @@ public:
                 // in well under a second. Cap it hard (independent of the much
                 // larger connect timeout) so an intermittently-dropped packet to
                 // 8.8.8.8 can't stall a connect for tens of seconds per host.
+                // Plain ternary, not std::min: <winsock2.h> defines a `min`
+                // macro that would mangle std::min on the (compiled-but-discarded)
+                // Windows branch of this if constexpr.
                 constexpr int kDnsTimeoutMs = 2500;
-                int dnsTimeout = timeoutMs > 0 ? std::min(timeoutMs, kDnsTimeoutMs)
-                                               : kDnsTimeoutMs;
+                int dnsTimeout = (timeoutMs > 0 && timeoutMs < kDnsTimeoutMs)
+                                     ? timeoutMs : kDnsTimeoutMs;
                 for (const auto& ip : platform::resolve_fallback(host, dnsTimeout)) {
                     if (try_resolved(ip.c_str(), /*numeric=*/true)) return true;
                 }