diff --git a/lib/googleauth.rb b/lib/googleauth.rb index c7cfe45..ca9b750 100644 --- a/lib/googleauth.rb +++ b/lib/googleauth.rb @@ -62,9 +62,14 @@ END json_key = MultiJson.load(json_key_io.read) fail "the json is missing the #{key} field" unless json_key.key?('type') type = json_key['type'] - return json_key, ServiceAccountCredentials if type == 'service_account' - return [json_key, UserRefreshCredentials] if type == 'authorized_user' - fail "credentials type '#{type}' is not supported" + case type + when 'service_account' + [json_key, ServiceAccountCredentials] + when 'authorized_user' + [json_key, UserRefreshCredentials] + else + fail "credentials type '#{type}' is not supported" + end end end @@ -82,9 +87,8 @@ END # @param scope [string|array|nil] the scope(s) to access # @param options [hash] allows override of the connection being used def get_application_default(scope = nil, options = {}) - creds = DefaultCredentials.from_env(scope) - return creds unless creds.nil? - creds = DefaultCredentials.from_well_known_path(scope) + creds = DefaultCredentials.from_env(scope) || + DefaultCredentials.from_well_known_path(scope) return creds unless creds.nil? fail NOT_FOUND_ERROR unless GCECredentials.on_gce?(options) GCECredentials.new diff --git a/spec/googleauth/signet_spec.rb b/spec/googleauth/signet_spec.rb index 60da4fd..884e2fe 100644 --- a/spec/googleauth/signet_spec.rb +++ b/spec/googleauth/signet_spec.rb @@ -45,8 +45,7 @@ describe Signet::OAuth2::Client do scope: 'https://www.googleapis.com/auth/userinfo.profile', issuer: 'app@example.com', audience: 'https://accounts.google.com/o/oauth2/token', - signing_key: @key - ) + signing_key: @key) end def make_auth_stubs(opts)