allow no ttl on local cache by setting ttl to max when config is -1 (#50)

emmanuelm41 · web-flow · commit 8b9a55b152b0 · 2023-12-28T15:50:14.000-03:00
diff --git a/pkg/zcache/config.go b/pkg/zcache/config.go
@@ -3,6 +3,7 @@ package zcache
 import (
 	"github.com/allegro/bigcache/v3"
 	"github.com/go-redis/redis/v8"
+	"math"
 	"time"
 )
 
@@ -47,7 +48,12 @@ func (c *RemoteConfig) ToRedisConfig() *redis.Options {
 }
 
 func (c *LocalConfig) ToBigCacheConfig() bigcache.Config {
-	return bigcache.DefaultConfig(time.Second * time.Duration(c.EvictionInSeconds))
+	eviction := time.Duration(c.EvictionInSeconds) * time.Second
+	if c.EvictionInSeconds < 0 {
+		eviction = time.Duration(math.MaxInt64)
+	}
+
+	return bigcache.DefaultConfig(eviction)
 }
 
 type CombinedConfig struct {

Original file line number	Diff line number	Diff line change
`@@ -3,6 +3,7 @@ package zcache`
`3`	`3`	`import (`
`4`	`4`	`"github.com/allegro/bigcache/v3"`
`5`	`5`	`"github.com/go-redis/redis/v8"`
	`6`	`+ "math"`
`6`	`7`	`"time"`
`7`	`8`	`)`
`8`	`9`
`@@ -47,7 +48,12 @@ func (c RemoteConfig) ToRedisConfig() redis.Options {`
`47`	`48`	`}`
`48`	`49`
`49`	`50`	`func (c *LocalConfig) ToBigCacheConfig() bigcache.Config {`
`50`		`- return bigcache.DefaultConfig(time.Second * time.Duration(c.EvictionInSeconds))`
	`51`	`+ eviction := time.Duration(c.EvictionInSeconds) * time.Second`
	`52`	`+ if c.EvictionInSeconds < 0 {`
	`53`	`+ eviction = time.Duration(math.MaxInt64)`
	`54`	`+ }`
	`55`	`+`
	`56`	`+ return bigcache.DefaultConfig(eviction)`
`51`	`57`	`}`
`52`	`58`
`53`	`59`	`type CombinedConfig struct {`