Skip to content

Commit

Permalink
fix / redis service.TTL() returns seconds, hasExpiration, found
Browse files Browse the repository at this point in the history
  • Loading branch information
akiraho committed Aug 16, 2018
1 parent 336d7c7 commit 89bef01
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions sessions/sessiondb/redis/service/service.go
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,6 @@ package service
import (
"fmt"
"time"

"github.com/gomodule/redigo/redis"
"github.com/kataras/iris/core/errors"
)
Expand Down Expand Up @@ -84,7 +83,7 @@ func (r *Service) Get(key string) (interface{}, error) {

// TTL returns the seconds to expire, if the key has expiration and error if action failed.
// Read more at: https://redis.io/commands/ttl
func (r *Service) TTL(key string) (seconds int64, hasExpiration bool, ok bool) {
func (r *Service) TTL(key string) (seconds int64, hasExpiration bool, found bool) {
c := r.pool.Get()
defer c.Close()
redisVal, err := c.Do("TTL", r.Config.Prefix+key)
Expand All @@ -93,14 +92,15 @@ func (r *Service) TTL(key string) (seconds int64, hasExpiration bool, ok bool) {
}
seconds = redisVal.(int64)
// if -1 means the key has unlimited life time.
hasExpiration = seconds == -1
hasExpiration = seconds > -1
// if -2 means key does not exist.
ok = (c.Err() != nil || seconds == -2)
found = ! (c.Err() != nil || seconds == -2)
return
}

func (r *Service) updateTTLConn(c redis.Conn, key string, newSecondsLifeTime int64) error {
reply, err := c.Do("EXPIRE", r.Config.Prefix+key, newSecondsLifeTime)

if err != nil {
return err
}
Expand Down

0 comments on commit 89bef01

Please sign in to comment.