Skip to content

Commit

Permalink
fix: fix python-api-core dependency issue (#931)
Browse files Browse the repository at this point in the history
`ClientOptions` doesn't have `client_encryted_cert_source` property yet. Check the existence of the property before accessing it.
  • Loading branch information
arithmetic1728 committed Jun 2, 2020
1 parent cafb9e0 commit 42028ed
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 2 deletions.
6 changes: 5 additions & 1 deletion googleapiclient/discovery.py
Expand Up @@ -441,7 +441,11 @@ def build_from_document(
raise MutualTLSChannelError(
"ClientOptions.client_cert_source is not supported, please use ClientOptions.client_encrypted_cert_source."
)
if client_options and client_options.client_encrypted_cert_source:
if (
client_options
and hasattr(client_options, "client_encrypted_cert_source")
and client_options.client_encrypted_cert_source
):
client_cert_to_use = client_options.client_encrypted_cert_source
elif adc_cert_path and adc_key_path and mtls.has_default_client_cert_source():
client_cert_to_use = mtls.default_client_encrypted_cert_source(
Expand Down
2 changes: 1 addition & 1 deletion setup.py
Expand Up @@ -41,7 +41,7 @@
"httplib2>=0.9.2,<1dev",
"google-auth>=1.16.0",
"google-auth-httplib2>=0.0.3",
"google-api-core>=1.13.0,<2dev",
"google-api-core>=1.17.0,<2dev",
"six>=1.6.1,<2dev",
"uritemplate>=3.0.0,<4dev",
]
Expand Down

0 comments on commit 42028ed

Please sign in to comment.