Fix default ports being rejected in preconnect

[heimskr]

What does this PR do?

Fixes an issue where port validation throws an error after default ports get normalized out of URLs passed to fetch.preconnect.

• Documentation or TypeScript types
• Code changes

How did you verify your code works?

I ran this code:

import { fetch } from "bun";
fetch.preconnect("https://example.com");

Fails in the current release version but doesn't with this PR.

[robobun]

^{Updated 7:37 PM PT - Jan 31st, 2025}

❌ @heimskr, your commit 2bc637c has 2 failures in Build #10948:

• test/js/bun/util/filesink.test.ts - timeout on 🪟 2019 x64-baseline
• test/js/bun/util/filesink.test.ts - timeout on 🪟 2019 x64
• test/cli/install/bun-install-registry.test.ts - 3 failing on 🪟 2019 x64

---

🧪   try this PR locally:

bunx bun-pr 16953

[heimskr]

Now it fails to reject port 0. I'll fix this.

[dylan-conway]

I don't fully understand this change yet, but similar might be needed in bun_js.zig in doPreconnect

[heimskr]

I don't fully understand this change yet, but similar might be needed in bun_js.zig in doPreconnect

My latest commit moves the check into hasValidPort, which is used in both doPreconnect and Bun__fetchPreconnect_.

[Jarred-Sumner]

Can you add a test