Merge pull request #204 from retail-ai-inc/fix/redis-dbname

fix tenant redis dbname issune
retail-ai-inc · Aug 20, 2024 · 3a1d253 · 3a1d253
2 parents 9e842e1 + 1ad57d2
commit 3a1d253
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 7 deletions.
diff --git a/bean.go b/bean.go
@@ -633,11 +633,7 @@ func (b *Bean) DefaultHTTPErrorHandler() echo.HTTPErrorHandler {
 		for _, handle := range b.errorHandlerFuncs {
 			handled, err := handle(err, c)
 			if err != nil {
-				if BeanConfig.Sentry.On {
-					SentryCaptureException(c, err)
-				} else {
-					c.Logger().Error(err)
-				}
+				SentryCaptureException(c, err)
 			}
 			if handled {
 				break
@@ -704,6 +700,7 @@ func SentryCaptureException(c echo.Context, err error) {
 	}
 
 	if !BeanConfig.Sentry.On {
+		Logger().Error(err)
 		return
 	}
 

diff --git a/internal/dbdrivers/redis.go b/internal/dbdrivers/redis.go
@@ -821,8 +821,8 @@ func getAllRedisTenantDB(config RedisConfig, tenantCfgs []*TenantConnections, te
 
 			port := redisCfg["port"].(string)
 			var dbName int
-			if dbName, ok = redisCfg["database"].(int); !ok {
-				dbName = 0
+			if _dbName, ok := redisCfg["database"].(float64); ok {
+				dbName = int(_dbName)
 			}
 
 			tenantRedisDB[t.TenantID] = &RedisDBConn{}