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

Perform more optimal GCM API credentials retrieval if available #116

Merged
merged 1 commit into from
Jul 8, 2024

Conversation

kzu
Copy link
Member

@kzu kzu commented Jul 8, 2024

The GCM binary retrieval may involve launching an auth UI which may not be available. By using the direct GCM API from the CredentialManager package, we can retrieve creds faster in most cases.

@kzu kzu added the enhancement New feature or request label Jul 8, 2024
We were not detecting non-empty directies (that contained only subdirectories), and we were also not cleaning up parent directories once the descendent ones were empty.

Perform more optimal GCM API credentials retrieval if available

The GCM binary retrieval may involve launching an auth UI which may not be available. By using the direct GCM API from the CredentialManager package, we can retrieve creds faster in most cases.
@kzu kzu enabled auto-merge (rebase) July 8, 2024 23:52
@kzu kzu merged commit c48bb1a into main Jul 8, 2024
8 checks passed
@kzu kzu deleted the dev/faster-creds branch July 8, 2024 23:53
@devlooped devlooped locked and limited conversation to collaborators Oct 19, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant