Variable inutilisée `expires` trouvée dans lib/secure_session.rb, cela semble être une erreur

coocooooo · Novembre 10, 2019, 6:48

Variable inutilisée expires trouvée dans lib/secure_session.rb, cela semble être une erreur

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 · Novembre 10, 2019, 1:19

@sam Est-ce que ça devrait être $redis.setex(prefixed_key(key), expires.to_i, val.to_s) ? Il semble que les tests ne échouent pas car ils utilisent la valeur par défaut de 1 heure.

sam · Novembre 11, 2019, 12:22

Bonne trouvaille, oui @gerhard, ça semble correct, c’est corrigé. Bonne trouvaille @coocooooo

Sujet		Réponses	Vues
A more efficient Redis lock Development	1	798	Janvier 31, 2021
Redis Error after update Self-hosting	9	2436	Mars 5, 2022
RSpec failure about redis Self-hosting	3	1122	Décembre 29, 2016
Sidekiq jobs are failing for 1 of my users Self-hosting	11	2181	Octobre 6, 2016
Requesting help with failure finding DiscourseRedis::Connector running tests Development	6	664	Juin 23, 2020

Variable inutilisée `expires` trouvée dans lib/secure_session.rb, cela semble être une erreur

Sujets connexes