diff --git a/gslib/tests/test_shim_util.py b/gslib/tests/test_shim_util.py index 70a02d0b0..3d39bfaa1 100644 --- a/gslib/tests/test_shim_util.py +++ b/gslib/tests/test_shim_util.py @@ -1251,7 +1251,7 @@ def test_gcs_json_endpoint_translation(self): self.assertEqual( env_vars, { 'CLOUDSDK_API_ENDPOINT_OVERRIDES_STORAGE': - 'https://foo_host:1234/storage/v2', + 'https://foo_host:1234/storage/v2/', }) def test_gcs_json_endpoint_translation_with_missing_port(self): @@ -1265,7 +1265,7 @@ def test_gcs_json_endpoint_translation_with_missing_port(self): self.assertEqual(flags, []) self.assertEqual(env_vars, { 'CLOUDSDK_API_ENDPOINT_OVERRIDES_STORAGE': - 'https://foo_host/storage/v2', + 'https://foo_host/storage/v2/', }) def test_gcs_json_endpoint_translation_usees_default_version_v1(self): @@ -1279,7 +1279,7 @@ def test_gcs_json_endpoint_translation_usees_default_version_v1(self): self.assertEqual( env_vars, { 'CLOUDSDK_API_ENDPOINT_OVERRIDES_STORAGE': - 'https://foo_host:1234/storage/v1' + 'https://foo_host:1234/storage/v1/' }) def test_s3_endpoint_translation(self): diff --git a/gslib/utils/shim_util.py b/gslib/utils/shim_util.py index c8b3c77db..aaad290ea 100644 --- a/gslib/utils/shim_util.py +++ b/gslib/utils/shim_util.py @@ -275,7 +275,7 @@ def _get_gcs_json_endpoint_from_boto_config(config): gs_json_port = config.get('Credentials', 'gs_json_port') port = ':' + gs_json_port if gs_json_port else '' json_api_version = config.get('Credentials', 'json_api_version', 'v1') - return 'https://{}{}/storage/{}'.format(gs_json_host, port, + return 'https://{}{}/storage/{}/'.format(gs_json_host, port, json_api_version) return None