在 lib/secure_session.rb 中发现未使用的变量 `expires`，这似乎是个错误

coocooooo · 2019 年11 月 10 日 06:48

在 lib/secure_session.rb 中发现未使用的变量 expires，这似乎是一个错误

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 · 2019 年11 月 10 日 13:19

@sam 是否应该是 $redis.setex(prefixed_key(key), expires.to_i, val.to_s)？看起来测试用例没有失败，是因为它们使用了默认的 1 小时值。

sam · 2019 年11 月 11 日 00:22

好眼力，是的 @gerhard，听起来没错，已修复。好眼力 @coocooooo

话题		回复	浏览量
A more efficient Redis lock Development	1	795	2021 年1 月 31 日
Redis Error after update Self-hosting	10	2421	2022 年4 月 4 日
RSpec failure about redis Self-hosting	4	1118	2016 年12 月 29 日
Sidekiq jobs are failing for 1 of my users Self-hosting	12	2167	2016 年11 月 16 日
Requesting help with failure finding DiscourseRedis::Connector running tests Development	7	661	2020 年6 月 23 日

在 lib/secure_session.rb 中发现未使用的变量 `expires`，这似乎是个错误

相关话题