Merge pull request #74185 from SimonSiefke/portmapping2

fix #74085

src/vs/vscode.d.ts

@@ -5763,8 +5763,11 @@ declare module 'vscode' {
 		 * to allow using a static localhost port inside the webview that is resolved to random port that a service is
 		 * running on.
 		 *
-		 * If a webview accesses localhost content, we recomend that you specify port mappings even if
+		 * If a webview accesses localhost content, we recommend that you specify port mappings even if
 		 * the `webviewPort` and `extensionHostPort` ports are the same.
+		 *
+		 * *Note* that port mappings only work for `http` or `https` urls. Websocket urls (e.g. `ws://localhost:3000`)
+		 * cannot be mapped to another port.
 		 */
 		readonly portMapping?: ReadonlyArray<WebviewPortMapping>;
 	}

src/vs/workbench/contrib/webview/electron-browser/webviewElement.ts

@@ -165,8 +165,7 @@ class WebviewPortMappingProvider extends Disposable {
 
 		session.onBeforeRequest(async (details) => {
 			const uri = URI.parse(details.url);
-			const allowedSchemes = ['http', 'https', 'ws', 'wss'];
-			if (allowedSchemes.indexOf(uri.scheme) === -1) {
+			if (uri.scheme !== 'http' && uri.scheme !== 'https') {
 				return undefined;
 			}