fix Host header in request correctly (#1004)

minio/api.py

@@ -182,11 +182,11 @@ def _handle_redirect_response(
 
         return code, message
 
-    def _build_headers(self, headers, body, creds):
+    def _build_headers(self, host, headers, body, creds):
         """Build headers with given parameters."""
         headers = headers or {}
         md5sum_added = headers.get("Content-MD5")
-        headers["Host"] = self._base_url.host
+        headers["Host"] = host
         headers["User-Agent"] = self._user_agent
         sha256 = None
         md5sum = None
@@ -226,14 +226,14 @@ def _url_open(  # pylint: disable=too-many-branches
         creds = self._provider.retrieve() if self._provider else None
         trace_body = isinstance(body, str)
         body = body.encode() if trace_body else body
-        headers, date = self._build_headers(headers, body, creds)
         url = self._base_url.build(
             method,
             region,
             bucket_name=bucket_name,
             object_name=object_name,
             query_params=query_params,
         )
+        headers, date = self._build_headers(url.netloc, headers, body, creds)
         if creds:
             headers = sign_v4_s3(
                 method,

minio/definitions.py

@@ -212,7 +212,7 @@ def build(
             (method == "PUT" and not object_name and not query_params) or
 
             # GetBucketLocation API requires path style in Amazon AWS S3.
-            (query_params and query_params.get("location")) or
+            (query_params and "location" in query_params) or
 
             # Use path style for bucket name containing '.' which causes
             # SSL certificate validation error.