eric-maynard opened a new pull request, #1237:
URL: https://github.com/apache/polaris/pull/1237
When running a command like this in the Polaris CLI:
```
./polaris \
--access-token 'principal:root;realm:default-realm' \
--client-id abc \
--client-secret xyz \
catalogs \
create test_catalog \
--storage-type file \
--default-base-location 'file:///tmp/polaris'
```
We reject the invocation because it's ambiguous whether the user would like
to use the access token or the client ID & secret.
However, it's also possible for the user to specify client ID & secret via
environment variables. In that case, we reject commands even like this:
```
./polaris \
--access-token 'principal:root;realm:default-realm' \
catalogs \
create test_catalog \
--storage-type file \
--default-base-location 'file:///tmp/polaris'
```
I believe that we should allow situations where the user explicitly provides
one set of credentials rather than failing due to the implicit inclusion of
another set of credentials via the environment variables.
--
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]
