feat(dev): support HTTP2 even if proxy feature is used

[Corvince]

Description

Fixes #4184. Previously using https and proxy server options did not work, since http-proxy didn't support https servers. With the recent switch to http-proxy3 this is no longer necessary and vite can support both options simultaneously, allowing to serve with http2.

[Corvince]

Realized this should result in a slight api change - the first argument to resolveHttpServer is of type CommonServerOptions - but only proxy is used here and with this PR no longer. So should the function signature of resolveHttpServer change, to remove the first argument? Not sure how to proceed.

[sapphi-red]

http-proxy-3 does not support HTTP2 (sagemathinc/http-proxy-3#2), so we cannot do this yet.

[sapphi-red]

http-proxy-3 now supports it (sagemathinc/http-proxy-3#33, #20925).

[sapphi-red]

Realized this should result in a slight api change - the first argument to resolveHttpServer is of type CommonServerOptions - but only proxy is used here and with this PR no longer. So should the function signature of resolveHttpServer change, to remove the first argument? Not sure how to proceed.

@Corvince Would you remove the first parameter from resolveHttpServer?

[Corvince]

Done. Let me know if there is anything else left to do.

[sapphi-red]

Thanks, looks good to me

[pkg-pr-new]

Open in StackBlitz

npm i https://pkg.pr.new/vite@20869

commit: 48da57b

[sapphi-red]

/ecosystem-ci run

[vite-ecosystem-ci]

📝 Ran ecosystem CI on a0fabd1: Open

suite	result	latest scheduled
laravel	❌ failure	❌ failure
qwik	❌ failure	❌ failure
react-router	✅ success	⏹️ cancelled

✅ ladle, rakkas, astro, analogjs, marko, histoire, nuxt, one, sveltekit, vite-plugin-pwa, storybook, quasar, vite-plugin-cloudflare, vite-setup-catalogue, vite-plugin-svelte, vitest, vitepress, vite-plugin-rsc, vite-environment-examples, vite-plugin-vue, waku, vite-plugin-react, unocss, vike, vuepress