fix(client): HTTP/2 should use hostname in URL as host instead of remoteHost

Author: fenying

CHANGES.md

@@ -1,5 +1,9 @@
 # Changes Logs
 
+## v1.0.4
+
+- fix(client): HTTP/2 should use hostname in URL as host instead of `remoteHost`
+
 ## v1.0.3
 
 - fix(client): Apply timeout for response stream.

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "@litert/http-client",
-  "version": "1.0.3",
+  "version": "1.0.4",
   "description": "A simple HTTP client based on LiteRT.",
   "main": "lib/index.js",
   "dependencies": {

package.json

@@ -206,15 +206,15 @@ export abstract class AbstractHttp2Client extends AbstractProtocolClient {
             [$H2.constants.HTTP2_HEADER_PATH]: this._.buildPath(opts.url)
         };
 
-        if (opts.connectionOptions.remoteHost) {
+        if (!headers[$H2.constants.HTTP2_HEADER_AUTHORITY]) {
 
-            opts.connectionOptions.servername = opts.url.hostname;
-            opts.url.hostname = opts.connectionOptions.remoteHost;
+            headers[$H2.constants.HTTP2_HEADER_AUTHORITY] = opts.url.hostname;
         }
 
-        if (!headers[$H2.constants.HTTP2_HEADER_AUTHORITY]) {
+        if (opts.connectionOptions.remoteHost) {
 
-            headers[$H2.constants.HTTP2_HEADER_AUTHORITY] = opts.url.hostname;
+            opts.connectionOptions.servername = opts.url.hostname;
+            opts.url.hostname = opts.connectionOptions.remoteHost;
         }
 
         const [connId, conn] = await this._getConnection(