coocooooo
10. November 2019 um 06:48
1
Unbenutzte Variable expires in lib/secure_session.rb gefunden, das scheint ein Fehler zu sein
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
10. November 2019 um 13:19
2
@sam Soll es $redis.setex(prefixed_key(key), expires.to_i, val.to_s) sein? Die Tests scheinen nicht zu fehlschlagen, da sie den Standardwert von 1 Stunde verwenden.
sam
11. November 2019 um 00:22
3
Toller Fund, ja @gerhard , das klingt richtig, behoben. Toller Fund @coocooooo
committed 12:18AM - 11 Nov 19 UTC
Previously we were always hard-coding expiry, this allows the secure session
to … correctly handle custom expiry times
Also adds a ttl method for looking up time to live