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

Create exception class for default credentials not found #380

Closed
wants to merge 1 commit into from

Conversation

bouk
Copy link
Contributor

@bouk bouk commented Apr 8, 2022

Part of #304

Potentially we should make Google::Auth::Error a subclass of RuntimeError so we don't break backwards compatibility?

@bouk bouk requested a review from a team as a code owner April 8, 2022 13:35
@viacheslav-rostovtsev
Copy link
Member

Thank you for submitting this PR.
The new error should definitely be a subclass of RuntimeError. That and fixing the rubocop errors from the CI should make it OK to merge.

https://developers.google.com/accounts/docs/application-default-credentials
for more information
ERROR_MESSAGE
class DefaultCredentialsNotFoundError < Error

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing docs for the new classes.

@NivedhaSenthil
Copy link
Member

@bouk gentle follow up to see if you are still looking at this ?

@bajajneha27
Copy link
Contributor

Hi @bouk
I'm closing the PR due to inactivity. Please feel free to reopen it if you still want to continue working on it.

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

Successfully merging this pull request may close these issues.

None yet

4 participants