fix(storage): update storage endpoint from www.googleapis.com to stor…

…age.googleapis.com (#9543)
googleapis · Oct 25, 2019 · 764251d · 764251d
1 parent 14e140c
commit 764251d
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 11 deletions.
diff --git a/storage/google/cloud/storage/_helpers.py b/storage/google/cloud/storage/_helpers.py
@@ -24,7 +24,7 @@
 STORAGE_EMULATOR_ENV_VAR = "STORAGE_EMULATOR_HOST"
 """Environment variable defining host for Storage emulator."""
 
-_DEFAULT_STORAGE_HOST = u"https://www.googleapis.com"
+_DEFAULT_STORAGE_HOST = u"https://storage.googleapis.com"
 
 
 def _get_storage_host():

diff --git a/storage/google/cloud/storage/_http.py b/storage/google/cloud/storage/_http.py
@@ -29,7 +29,7 @@ class Connection(_http.JSONConnection):
     :param client_info: (Optional) instance used to generate user agent.
     """
 
-    DEFAULT_API_ENDPOINT = _http.API_BASE_URL
+    DEFAULT_API_ENDPOINT = "https://storage.googleapis.com"
 
     def __init__(self, client, client_info=None, api_endpoint=DEFAULT_API_ENDPOINT):
         super(Connection, self).__init__(client, client_info)

diff --git a/storage/tests/unit/test_blob.py b/storage/tests/unit/test_blob.py
@@ -725,7 +725,7 @@ def test__get_download_url_on_the_fly(self):
         self.assertIsNone(blob.media_link)
         download_url = blob._get_download_url()
         expected_url = (
-            "https://www.googleapis.com/download/storage/v1/b/"
+            "https://storage.googleapis.com/download/storage/v1/b/"
             "buhkit/o/bzzz-fly.txt?alt=media"
         )
         self.assertEqual(download_url, expected_url)
@@ -741,7 +741,7 @@ def test__get_download_url_on_the_fly_with_generation(self):
         self.assertIsNone(blob.media_link)
         download_url = blob._get_download_url()
         expected_url = (
-            "https://www.googleapis.com/download/storage/v1/b/"
+            "https://storage.googleapis.com/download/storage/v1/b/"
             "fictional/o/pretend.txt?alt=media&generation=1493058489532987"
         )
         self.assertEqual(download_url, expected_url)
@@ -755,7 +755,7 @@ def test__get_download_url_on_the_fly_with_user_project(self):
         self.assertIsNone(blob.media_link)
         download_url = blob._get_download_url()
         expected_url = (
-            "https://www.googleapis.com/download/storage/v1/b/"
+            "https://storage.googleapis.com/download/storage/v1/b/"
             "fictional/o/pretend.txt?alt=media&userProject={}".format(user_project)
         )
         self.assertEqual(download_url, expected_url)
@@ -774,7 +774,7 @@ def test__get_download_url_on_the_fly_with_kms_key_name(self):
         self.assertIsNone(blob.media_link)
         download_url = blob._get_download_url()
         expected_url = (
-            "https://www.googleapis.com/download/storage/v1/b/"
+            "https://storage.googleapis.com/download/storage/v1/b/"
             "buhkit/o/bzzz-fly.txt?alt=media"
         )
         self.assertEqual(download_url, expected_url)
@@ -1020,7 +1020,7 @@ def test_download_to_file_wo_media_link(self):
         self.assertIsNone(blob.media_link)
 
         expected_url = (
-            "https://www.googleapis.com/download/storage/v1/b/"
+            "https://storage.googleapis.com/download/storage/v1/b/"
             "name/o/blob-name?alt=media"
         )
         self._check_session_mocks(client, transport, expected_url)
@@ -1375,7 +1375,9 @@ def _do_multipart_success(
 
         mock_get_boundary.assert_called_once_with()
 
-        upload_url = "https://www.googleapis.com/upload/storage/v1" + bucket.path + "/o"
+        upload_url = (
+            "https://storage.googleapis.com/upload/storage/v1" + bucket.path + "/o"
+        )
 
         qs_params = [("uploadType", "multipart")]
 
@@ -1498,7 +1500,9 @@ def _initiate_resumable_helper(
         # Check the returned values.
         self.assertIsInstance(upload, ResumableUpload)
 
-        upload_url = "https://www.googleapis.com/upload/storage/v1" + bucket.path + "/o"
+        upload_url = (
+            "https://storage.googleapis.com/upload/storage/v1" + bucket.path + "/o"
+        )
         qs_params = [("uploadType", "resumable")]
 
         if user_project is not None:
@@ -1621,7 +1625,7 @@ def _make_resumable_transport(self, headers1, headers2, headers3, total_bytes):
     def _do_resumable_upload_call0(blob, content_type, size=None, predefined_acl=None):
         # First mock transport.request() does initiates upload.
         upload_url = (
-            "https://www.googleapis.com/upload/storage/v1"
+            "https://storage.googleapis.com/upload/storage/v1"
             + blob.bucket.path
             + "/o?uploadType=resumable"
         )
@@ -1980,7 +1984,7 @@ def _create_resumable_upload_session_helper(self, origin=None, side_effect=None)
 
         # Check the mocks.
         upload_url = (
-            "https://www.googleapis.com/upload/storage/v1"
+            "https://storage.googleapis.com/upload/storage/v1"
             + bucket.path
             + "/o?uploadType=resumable"
         )