remove that port from the pattern (when is that ever useful?)

Signed-off-by: Michael Sprauer <[email protected]>

Author: MichaelSp

lib/middleware/serveProxies.js

@@ -1,4 +1,4 @@
-// eavily inspired by https://github.com/SAP/connect-openui5/blob/master/lib/proxy.js
+// heavily inspired by https://github.com/SAP/connect-openui5/blob/master/lib/proxy.js
 const url = require("url");
 const httpProxy = require("http-proxy");
 
@@ -9,14 +9,18 @@ const env = {
 };
 
 function getProxyUri(uri) {
+	console.log("1", env)
 	if (uri.protocol === "https:" && env.httpsProxy || uri.protocol === "http:" && env.httpProxy) {
-		if (env.noProxy) {
+		console.log("2")
+		if (env.noProxy) {console.log("3")
 			const canonicalHost = uri.host.replace(/^\.*/, ".");
 			const port = uri.port || (uri.protocol === "https:" ? "443" : "80");
 
 			const patterns = env.noProxy.split(",");
 			for (let i = patterns.length - 1; i >= 0; i--) {
+				console.log("4")
 				let pattern = patterns[i].trim().toLowerCase();
+				console.log(canonicalHost, pattern)
 
 				// don"t use a proxy at all
 				if (pattern === "*") {
@@ -26,11 +30,7 @@ function getProxyUri(uri) {
 				// Remove leading * and make sure to have exact one leading dot (.)
 				pattern = pattern.replace(/^[*]+/, "").replace(/^\.*/, ".");
 
-				// add port if no specified
-				if (pattern.indexOf(":") === -1) {
-					pattern += ":" + port;
-				}
-
+				console.log(canonicalHost, pattern)
 				// if host ends with pattern, no proxy should be used
 				if (canonicalHost.indexOf(pattern) === canonicalHost.length - pattern.length) {
 					return null;