[ReverseProxyPlugin] Remove redundant ca_file flag when wrapping upstream (#1046)

* Fix #1045

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* no conn log

* Do not edit host header by default

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

Author: abhinavsingh

proxy/core/connection/server.py

@@ -43,7 +43,7 @@ def connect(
         )
         self.closed = False
 
-    def wrap(self, hostname: str, ca_file: Optional[str]) -> None:
+    def wrap(self, hostname: str, ca_file: Optional[str] = None) -> None:
         ctx = ssl.create_default_context(
             ssl.Purpose.SERVER_AUTH, cafile=ca_file,
         )

proxy/http/server/reverse.py

@@ -20,7 +20,7 @@
 from ...core.base import TcpUpstreamConnectionHandler
 from ...common.utils import text_
 from ...common.constants import (
-    DEFAULT_HTTP_PORT, DEFAULT_HTTPS_PORT,
+    HTTPS_PROTO, DEFAULT_HTTP_PORT, DEFAULT_HTTPS_PORT,
     DEFAULT_REVERSE_PROXY_ACCESS_LOG_FORMAT,
 )
 
@@ -78,12 +78,18 @@ def handle_request(self, request: HttpParser) -> None:
         assert self.upstream
         try:
             self.upstream.connect()
-            if self.choice.scheme == b'https':
+            if self.choice.scheme == HTTPS_PROTO:
                 self.upstream.wrap(
                     text_(
                         self.choice.hostname,
-                    ), ca_file=str(self.flags.ca_file),
+                    ),
                 )
+            # Update Host header
+            # if request.has_header(b'Host'):
+            #     request.del_header(b'Host')
+            # request.add_header(
+            #     b'Host', ('%s:%d' % self.upstream.addr).encode('utf-8'),
+            # )
             self.upstream.queue(memoryview(request.build()))
         except ConnectionRefusedError:
             raise HttpProtocolException(