oauth rubocop aufzuräumen Unterstützung

Question

ich die folgende Methode Setup mit erfrischenden oauth Token zu unterstützen:

module WhiplashOmniAuthentication 
    extend ActiveSupport::Concern 

    module ClassMethods 
    def from_omniauth(auth) 
     Rails.logger.debug auth.inspect 
     where(provider: auth.provider, uid: auth.uid).first_or_create do |user| 
     user.provider = auth.provider 
     user.uid = auth.uid 
     user.email = auth.info.email 
     user.store_token(auth.credentials) 
     end 
    end 
    end 

    def refresh_token! 
    settings = Devise.omniauth_configs[:whiplash].strategy 
    strategy = OmniAuth::Strategies::Whiplash.new(nil, settings.client_id, settings.client_secret, client_options: settings.client_options) 
    client = strategy.client 
    access_token = OAuth2::AccessToken.new client, token, refresh_token: refresh_token 
    if access_token 
     begin 
     result = access_token.refresh! 
     store_token(result) 
     save 
     rescue OAuth2::Error => e 
     errors[:token] << e.inspect 
     return false 
     end 
    else 
     errors[:token] << e.inspect 
     return false 
    end 
    end 

    def store_token(auth_token) 
    self.token = auth_token.token 
    self.refresh_token = auth_token.refresh_token 
    self.token_expires_at = Time.at(auth_token.expires_at).to_datetime 
    end 

    def token_expired? 
    Time.now > token_expires_at 
    end 
end 

ich schon versucht, diese zu brechen in einzelne Methoden, aber es hält Sprengung, so werde ich die aufzuschieben SO Götter hier. Ich suche nach Empfehlungen, um die Polizei zu übergeben und zu lernen. Danke

Answer 1

Sie haben definitiv zu viele Dinge in der refresh_token! Implementierung. Sie wollen immer Methoden beibehalten, um eine Sache und nur eine Sache zu tun. Es erleichtert das Testen (z. B. Ausgeben einer bestimmten Methode), Debugging und Lesbarkeit.

Prüfen Sie, ob die folgenden hilft:

module WhiplashOmniAuthentication 
    extend ActiveSupport::Concern 

    module ClassMethods 
    def from_omniauth(auth) 
     Rails.logger.debug auth.inspect 
     where(provider: auth.provider, uid: auth.uid).first_or_create do |user| 
     user.provider = auth.provider 
     user.uid = auth.uid 
     user.email = auth.info.email 
     user.store_token(auth.credentials) 
     end 
    end 
    end 

    def refresh_token! 
    access_token ? refresh_access_token! : false 
    end 

    def refresh_access_token! 
    result = access_token.refresh! 
    store_token(result) 
    save 
    rescue OAuth2::Error 
    false 
    end 

    def settings 
    @settings ||= Devise.omniauth_configs[:whiplash].strategy 
    end 

    def strategy 
    @strategy ||= OmniAuth::Strategies::Whiplash.new(nil, settings.client_id, settings.client_secret, client_options: settings.client_options) 
    end 

    def client 
    @client ||= strategy.client 
    end 

    def access_token 
    OAuth2::AccessToken.new(client, token, refresh_token: refresh_token) 
    end 

    def store_token(auth_token) 
    self.token = auth_token.token 
    self.refresh_token = auth_token.refresh_token 
    self.token_expires_at = Time.at(auth_token.expires_at).to_datetime 
    end 

    def token_expired? 
    Time.now > token_expires_at 
    end 
end