ERR value is not an integer or out of range

[armolee]

Ask questions at https://stackoverflow.com/questions/ask?tags=go+redis

func init() {
    pool = &redis2.Pool{
        MaxIdle:     32,
        MaxActive:   0,
        IdleTimeout: time.Duration(120),
        Dial: func() (redis2.Conn, error) {
            return redis2.Dial(
                "tcp",
                global.RedisServer+":"+global.RedisPort,
                redis2.DialReadTimeout(time.Duration(1000)*time.Millisecond),
                redis2.DialWriteTimeout(time.Duration(1000)*time.Millisecond),
                redis2.DialConnectTimeout(time.Duration(1000)*time.Millisecond),
            )
        },
    }
}

func Set(key, value string, timeout int) (err error) {
    c := pool.Get()
    defer func() {
        _ = c.Close()
    }()
    _, err = c.Do("set", key, value, "EX", time.Duration(timeout)*time.Second)
    if err != nil {
        return errors.New(fmt.Sprintf("set %s=%s failed, err=%v", key, value, err))
    }
    return err
}

in the code:

err = Set("xxx-xxx-robot-token","at_512fd330-f287-48ae-90c9-6f2545849110", 3600)

2022/05/19 22:11:54.089 [W] [send.go:84] set robot token to redis failed, err = set xxx-xxx-robot-token=at_512fd330-f287-48ae-90c9-6f2545849110 failed, err=ERR value is not an integer or out of range

in redis cli:

[stevenh]

EX value is in seconds so I suspect you want the following which also includes some style cleanups

func Set(key, value string, timeout int) error {
    c := pool.Get()
    defer c.Close()

    if _, err := c.Do("set", key, value, "EX", timeout); err != nil {
        return fmt.Errorf("set %s=%s failed, err=%w", key, value, err)
    }
    return nil
}