Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

object store ApplicationDefaultCredentials auth is not working on windows #5466

Closed
itay-jether opened this issue Mar 4, 2024 · 3 comments · Fixed by #5473
Closed

object store ApplicationDefaultCredentials auth is not working on windows #5466

itay-jether opened this issue Mar 4, 2024 · 3 comments · Fixed by #5473
Labels

Comments

@itay-jether
Copy link
Contributor

Object store default authentication (ApplicationDefaultCredentials) works with looks for a json file, which on linusx is on usually on ~/.config/gcloud/application_default_credentials.json. The path is determined by using concatentating the path of HOME environment variable with the string .config/gcloud/application_default_credentials.json

However, on windows it can be in other paths, like C:\Users\[User]\AppData\Roaming\gcloud directory, the directroy can be retrieved by gcloud info, under 'User Config Directory'.

@tustvold
Copy link
Contributor

tustvold commented Mar 4, 2024

I'm afraid I don't have a windows machine on which to test this, but would welcome a PR. I presume this information must be encoded in the environment somehow

@Xuanwo
Copy link
Member

Xuanwo commented Mar 5, 2024

However, on windows it can be in other paths, like C:\Users\[User]\AppData\Roaming\gcloud directory, the directroy can be retrieved by gcloud info, under 'User Config Directory'.

On windows, we should read %APPDATA%\gcloud\application_default_credentials.json instead.

@tustvold
Copy link
Contributor

label_issue.py automatically added labels {'object-store'} from #5473

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants