feat(log): warn users if they use the default cache

darkweak · Jul 7, 2024 · d363dd2 · d363dd2
1 parent 84b1908
commit d363dd2
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 6 deletions.
diff --git a/pkg/middleware/middleware.go b/pkg/middleware/middleware.go
@@ -66,6 +66,7 @@ func NewHTTPCacheHandler(c configurationtypes.AbstractConfigurationInterface) *S
 	storedStorers := core.GetRegisteredStorers()
 	storers := make([]types.Storer, len(storedStorers))
 	if len(storedStorers) == 0 {
+		c.GetLogger().Warn("You're running Souin with the default storage that is not optimized and for development purpose. We recommend to use at least one of the storages from https://github.com/darkweak/storages")
 		memoryStorer, _ := storage.Factory(c)
 		core.RegisterStorage(memoryStorer)
 		storers = append(storers, memoryStorer)

diff --git a/plugins/caddy/httpcache.go b/plugins/caddy/httpcache.go
@@ -246,12 +246,6 @@ func (s *SouinCaddyMiddleware) Provision(ctx caddy.Context) error {
 		return err
 	}
 
-	bh := middleware.NewHTTPCacheHandler(&s.Configuration)
-	surrogates, ok := up.LoadOrStore(surrogate_key, bh.SurrogateKeyStorer)
-	if ok {
-		bh.SurrogateKeyStorer = surrogates.(surrogates_providers.SurrogateInterface)
-	}
-
 	if s.Configuration.DefaultCache.Badger.Found {
 		e := dispatchStorage(ctx, "badger", s.Configuration.DefaultCache.Badger, s.Configuration.DefaultCache.GetStale())
 		if e != nil {
@@ -289,6 +283,12 @@ func (s *SouinCaddyMiddleware) Provision(ctx caddy.Context) error {
 		}
 	}
 
+	bh := middleware.NewHTTPCacheHandler(&s.Configuration)
+	surrogates, ok := up.LoadOrStore(surrogate_key, bh.SurrogateKeyStorer)
+	if ok {
+		bh.SurrogateKeyStorer = surrogates.(surrogates_providers.SurrogateInterface)
+	}
+
 	s.SouinBaseHandler = bh
 	if len(app.Storers) == 0 {
 		app.Storers = s.SouinBaseHandler.Storers