lib/secure_session.rb で未使用の変数 `expires` が見つかりました。これは誤りのようです。

coocooooo · 2019 年 11 月 10 日午前 6: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 日午後 1:19

@sam $redis.setex(prefixed_key(key), expires.to_i, val.to_s) でしょうか？仕様テストが失敗していないのは、デフォルト値の 1 時間を使用しているためのようです。

sam · 2019 年 11 月 11 日午前 12:22

見つけられてよかった、その通りだね @gerhard。修正しました。見つけてくれてありがとう @coocooooo

トピック		返信	表示
A more efficient Redis lock Development	1	798	2021 年 1 月 31 日
Redis Error after update Self-hosting	9	2436	2022 年 3 月 5 日
RSpec failure about redis Self-hosting	3	1122	2016 年 12 月 29 日
Sidekiq jobs are failing for 1 of my users Self-hosting	11	2181	2016 年 10 月 6 日
Requesting help with failure finding DiscourseRedis::Connector running tests Development	6	664	2020 年 6 月 23 日

lib/secure_session.rb で未使用の変数 `expires` が見つかりました。これは誤りのようです。

関連トピック