fix(turbopack): Use correct method for the proxy of fetch

[kdy1]

Description

We should use Proxy::all instead of Proxy::http or Proxy::https.

Closes PACK-2448

Testing Instructions

See next.js counterpart: vercel/next.js#63309

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
examples-basic-web	🔄 Building (Inspect)	Visit Preview	💬 Add feedback	Apr 1, 2024 6:01am
examples-nonmonorepo	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Apr 1, 2024 6:01am
examples-svelte-web	🔄 Building (Inspect)	Visit Preview	💬 Add feedback	Apr 1, 2024 6:01am
examples-tailwind-web	🔄 Building (Inspect)	Visit Preview	💬 Add feedback	Apr 1, 2024 6:01am

6 Ignored Deployments

Name	Status	Preview	Comments	Updated (UTC)
examples-designsystem-docs	⬜️ Ignored (Inspect)	Visit Preview		Apr 1, 2024 6:01am
examples-gatsby-web	⬜️ Ignored (Inspect)	Visit Preview		Apr 1, 2024 6:01am
examples-kitchensink-blog	⬜️ Ignored (Inspect)	Visit Preview		Apr 1, 2024 6:01am
examples-native-web	⬜️ Ignored (Inspect)	Visit Preview		Apr 1, 2024 6:01am
examples-vite-web	⬜️ Ignored (Inspect)	Visit Preview		Apr 1, 2024 6:01am
rust-docs	⬜️ Ignored (Inspect)	Visit Preview		Apr 1, 2024 6:01am

[kwonoj]

I tried this but it still didn't pass the test in next.js - did you able to make it actually work?

[github-actions]

🟢 Turbopack Benchmark CI successful 🟢

Thanks

[github-actions]

✅ This change can build next-swc

[github-actions]

🟢 CI successful 🟢

Thanks

[kdy1]

Actually nope, I think it's a bug of reqwest. It does not work as documented.

[kdy1]

It works if I add

let url = url.replace("https://", "http://");

so I think reqwest is not sending https requests to the proxy, regardless of the method.

[kwonoj]

That's what I saw as well (https://vercel.slack.com/archives/C03EWR7LGEN/p1707871876798169?thread_ts=1707788191.248449&cid=C03EWR7LGEN)

[kdy1]

Closing in favor of #8165