only set server.proxy when server.https is present

Conduitry · Conduitry · commit 9b01fd476966 · 2022-01-26T15:30:45.000-05:00
diff --git a/packages/kit/src/core/dev/index.js b/packages/kit/src/core/dev/index.js
@@ -89,7 +89,7 @@ export async function dev({ cwd, port, host, https, config }) {
 	// however, node-fetch's Request implementation does not like the HTTP/2 headers
 	// we set a no-op proxy config to force Vite to downgrade to TLS-only
 	// see https://vitejs.dev/config/#server-https
-	if (!merged_config.server.proxy) {
+	if (merged_config.server.https && !merged_config.server.proxy) {
 		merged_config.server.proxy = {};
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -89,7 +89,7 @@ export async function dev({ cwd, port, host, https, config }) {`
`89`	`89`	`// however, node-fetch's Request implementation does not like the HTTP/2 headers`
`90`	`90`	`// we set a no-op proxy config to force Vite to downgrade to TLS-only`
`91`	`91`	`// see https://vitejs.dev/config/#server-https`
`92`		`- if (!merged_config.server.proxy) {`
	`92`	`+ if (merged_config.server.https && !merged_config.server.proxy) {`
`93`	`93`	`merged_config.server.proxy = {};`
`94`	`94`	`}`
`95`	`95`