lib/secure_session.rb에서 사용되지 않는 변수 `expires` 발견, 실수인 것 같습니다

coocooooo · 11월 10, 2019, 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 · 11월 10, 2019, 1:19오후

@sam $redis.setex(prefixed_key(key), expires.to_i, val.to_s)가 맞나요? 스펙이 실패하지 않는 이유는 기본값인 1시간을 사용하고 있기 때문인 것 같습니다.

sam · 11월 11, 2019, 12:22오전

좋은 지적이에요, 네 @gerhard 말씀하신 대로 맞아요. 수정했습니다. 좋은 지적이에요 @coocooooo

글		댓글	조회수
A more efficient Redis lock Development	1	823	1월 31, 2021
Redis Error after update Self-hosting	9	2561	3월 5, 2022
RSpec failure about redis Self-hosting	3	1192	12월 29, 2016
Sidekiq jobs are failing for 1 of my users Self-hosting	11	2310	10월 6, 2016
Requesting help with failure finding DiscourseRedis::Connector running tests Development	6	734	6월 23, 2020

lib/secure_session.rb에서 사용되지 않는 변수 `expires` 발견, 실수인 것 같습니다

Related topics