[ADAP-783] [Feature] Allow user to be omitted if using oauth authenticator #726
Labels
good_first_issue
Good for newcomers
help_wanted
Extra attention is needed
type:enhancement
New feature or request
Is this your first time submitting a feature request?
Describe the feature
It would be nice if the
user
field was not mandatory when usingauthenticator: oauth
. Without specifying it, I get following error:When specifying it, it means I have to match the username that'd be fetched for oauth, which is a bit redundant, and makes the authenticator type a bit more annoying to use where when constructing a profile for local usage, we're authenticating with snowflake oauth to get the refresh token, and then using that token to run a single query to get the current username which we then use to fully construct the
profiles.yml
file.Describe alternatives you've considered
No real alternative other than having to specify username as far as I'm aware. Specifying a "dummy" value gives me the following error:
An alternative to making it totally optional would be to allow some way to specify a dummy value that's not passed to
snowflake.connector.connect
.Who will this benefit?
People trying to use the oauth authenticator type
Are you interested in contributing this feature?
yes
Anything else?
We had initially tried configuring the profile based on the comment at #474 (comment), where they don't include the username.
The text was updated successfully, but these errors were encountered: