flyrain commented on code in PR #1237:
URL: https://github.com/apache/polaris/pull/1237#discussion_r2008338057
##########
client/python/cli/polaris_cli.py:
##########
@@ -118,7 +118,7 @@ def _get_client_builder(options):
# Validates
has_access_token = options.access_token is not None
has_client_secret = client_id is not None and client_secret is not None
- if has_access_token and has_client_secret:
+ if has_access_token and (options.client_id or options.client_secret):
Review Comment:
Not a blocker: i'm OK even both are configured. We could let it takes token
first, if not found, then try to find client id and secrets. Only failed when
neither is available.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
