Encontrada variável não utilizada `expires` em lib/secure_session.rb, parece ser um erro

coocooooo · Novembro 10, 2019, 6:48am

Encontrada variável não utilizada expires em lib/secure_session.rb, parece ser um erro

lib/secure_session.rb

15f6f57cd


      
          
          def self.expiry
            @expiry ||= 1.hour.to_i
          end
          
          def self.expiry=(val)
            @expiry = val
          end
          
          def set(key, val, expires: nil)
            expires ||= SecureSession.expiry
            $redis.setex(prefixed_key(key), SecureSession.expiry.to_i, val.to_s)
            true
          end
          
          def [](key)
            $redis.get(prefixed_key(key))
          end
          
          def []=(key, val)
            if val == nil

def set(key, val, expires: nil)
    expires ||= SecureSession.expiry
    $redis.setex(prefixed_key(key), SecureSession.expiry.to_i, val.to_s)
    true
  end

gerhard · Novembro 10, 2019, 1:19pm

@sam Deveria ser $redis.setex(prefixed_key(key), expires.to_i, val.to_s)? Parece que os testes não falham porque usam o valor padrão de 1 hora.

sam · Novembro 11, 2019, 12:22am

Bom achado, sim @gerhard, parece correto, corrigido. Bom achado @coocooooo

Tópico		Respostas	Visualizações
A more efficient Redis lock Development	1	798	31 de Janeiro de 2021
Redis Error after update Self-hosting	9	2436	5 de Março de 2022
RSpec failure about redis Self-hosting	3	1122	29 de Dezembro de 2016
Sidekiq jobs are failing for 1 of my users Self-hosting	11	2179	6 de Outubro de 2016
Requesting help with failure finding DiscourseRedis::Connector running tests Development	6	664	23 de Junho de 2020

Encontrada variável não utilizada `expires` em lib/secure_session.rb, parece ser um erro

Tópicos relacionados