regression bug with parse-url update

This commit is contained in:
Dave Horton
2022-09-20 09:31:44 +02:00
parent 0c089e2380
commit 05c5d2211f

View File

@@ -28,8 +28,8 @@ class HttpRequestor extends BaseRequestor {
assert(['GET', 'POST'].includes(this.method)); assert(['GET', 'POST'].includes(this.method));
const u = this._parsedUrl = parseUrl(this.url); const u = this._parsedUrl = parseUrl(this.url);
this._baseUrl = `${u.protocol}://${u.resource}:${u.port || 80}`; if (u.port) this._baseUrl = `${u.protocol}://${u.resource}:${u.port}`;
this._resource = u.resource; else this._baseUrl = `${u.protocol}://${u.resource}`;
this._protocol = u.protocol; this._protocol = u.protocol;
this._search = u.search; this._search = u.search;
this._usePools = process.env.HTTP_POOL && parseInt(process.env.HTTP_POOL); this._usePools = process.env.HTTP_POOL && parseInt(process.env.HTTP_POOL);
@@ -49,7 +49,10 @@ class HttpRequestor extends BaseRequestor {
this.logger.debug(`HttpRequestor:created pool for ${this._baseUrl}`); this.logger.debug(`HttpRequestor:created pool for ${this._baseUrl}`);
} }
} }
else this.client = new Client(`${u.protocol}://${u.resource}:${u.port || 80}`); else {
if (u.port) this.client = new Client(`${u.protocol}://${u.resource}:${u.port}`);
else this.client = new Client(`${u.protocol}://${u.resource}`);
}
} }
get baseUrl() { get baseUrl() {
@@ -101,7 +104,8 @@ class HttpRequestor extends BaseRequestor {
query = u.query; query = u.query;
} }
else { else {
client = newClient = new Client(`${u.protocol}://${u.resource}:${u.port || 80}`); if (u.port) client = newClient = new Client(`${u.protocol}://${u.resource}:${u.port}`);
else client = newClient = new Client(`${u.protocol}://${u.resource}`);
path = u.pathname; path = u.pathname;
query = u.query; query = u.query;
} }