fix: Improve set_port validation and get_host behavior

src/ada_c.cpp

@@ -290,7 +290,19 @@ bool ada_set_port(ada_url result, const char* input, size_t length) noexcept {
   if (!r) {
     return false;
   }
-  return r->set_port(std::string_view(input, length));
+
+  std::string_view port_input(input, length);
+
+  if (port_input.find_first_not_of("0123456789") != std::string_view::npos) {
+    return false;
+  }
+
+  int port = std::stoi(std::string(port_input));
+  if (port < 1 || port > 65535) {
+    return false;
+  }
+
+  return r->set_port(port_input);
 }
 
 bool ada_set_pathname(ada_url result, const char* input,

src/url_aggregator.cpp

@@ -298,6 +298,13 @@ bool url_aggregator::set_port(const std::string_view input) {
   // Revert changes if parse_port fails.
   uint32_t previous_port = components.port;
   parse_port(trimmed);
+
+  std::string port_value = std::string(trimmed);
+  int port = std::stoi(port_value);
+  if (port < 1 || port > 65535) {
+    return false;
+  }
+
   if (is_valid) {
     return true;
   }

tests/basic_tests.cpp

@@ -17,6 +17,7 @@ TYPED_TEST(basic_tests, insane_url) {
   ASSERT_EQ(r->get_hostname(), "");
   ASSERT_EQ(r->get_port(), "");
   ASSERT_EQ(r->get_pathname(), "@EEEEEEEEEE");
+  ASSERT_FALSE(r->set_port("invalid80"));
   SUCCEED();
 }