HMGet can't scan type time.Time

[ryan961]

Version: github.com/go-redis/redis/v8 v8.11.4

Hi, I found the test case(https://github.com/go-redis/redis/blob/e061db8c13fd6b5e007d73370873026e360719de/redis_test.go#L274)，`time.Time` can be scanned to object. But it's wrong when I used it in hmget(...).scan(&u)

type User struct {
	Name string `json:"name" redis:"name"`
	Age  int64  `json:"age" redis:"age"`
	CreateAt time.Time `json:"createAt" redis:"createAt"`
}

func main() {
                ......


		fmt.Println(cli.HMSet(ctx, "joe", map[string]interface{}{
			"name":     "joe",
			"age":      10,
			"createAt": time.Now(),
		}).Result())

		u := User{}
		fmt.Println(cli.HMGet(ctx, "joe", "name", "age", "city", "createAt").Scan(&u))
		fmt.Println(u)
                
                ......
}

[edoger]

Ah, this library also has this feature! Map the data in the hash table to the structure!

[ntswamp]

@edoger
what are you saying dude?

[edoger]

what are you saying dude?

@ntswamp I never knew there was such a feature that we implemented a similar one ourselves!

[leoantony72]

Any update on this issue?

[michelemendel]

I also have issues with this. Reading the data from redis-cli, the date is correct.

Btw, according to
https://github.com/redis/go-redis/blob/master/commands_test.go#L1324
CreateAt should use redis:"time"

[ryan961]

I followed the author's update on Scanner interface and It solves the hmget scan problem perfectly. But the hset struct field (type TimeValue) is not successful.

parsing time "\x01\x00\x00\x00\x0e\xdb\xa4\xaf\xe7\x14\xf4\xe6\x10\x01\xe0" as "2006-01-02T15:04:05Z07:00": cannot parse "\x01\x00\x00\x00\x0e\xdb\xa4\xaf\xe7\x14\xf4\xe6\x10\x01\xe0" as "2006"

Does the author will add functions similar to scanner interface in the future?

func (t *TimeValue) RedisValue() string {
	return t.Time.String()
}

[monkey92t]

If time. Time:

type MyStruct struct {
        // call:time.UnmarshalText(data) , parseRFC3339
        Time time.Time `redis:"time"`
}

Custom time format:

type TimeValue struct {
	time.Time
}

func (t *TimeValue) ScanRedis(s string) (err error) {
	t.Time, err = time.Parse(time.Date, s)
	return
}

type data struct {
        Key1 string    `redis:"key1"`
        Key2 int       `redis:"key2"`
        Time TimeValue `redis:"time"`
}

var d data
res := client.MGet(ctx, "key1", "key2", "time")
res.Scan(&d)

[ndyakov]

This can be either resolved as @monkey92t suggested or @michelemendel, you can use newer version. This should be supported right now.