RHINENG-22333: compare config values not pointers to config

Author: MichaelMraka

base/database/setup.go

@@ -16,30 +16,30 @@ var (
 	DBReadReplica             *gorm.DB
 	OtherAdvisoryTypes        []string
 	AdvisoryTypes             map[int]string
-	globalPgConfig            *PostgreSQLConfig
-	globalPgReadReplicaConfig *PostgreSQLConfig
+	globalPgConfig            PostgreSQLConfig
+	globalPgReadReplicaConfig PostgreSQLConfig
 )
 
-func initDB(db *gorm.DB, pgConfig *PostgreSQLConfig, globalPgConfig *PostgreSQLConfig) {
-	if db == nil || pgConfig != globalPgConfig {
+func initDB(db *gorm.DB, pgConfig PostgreSQLConfig, globalPgConfig *PostgreSQLConfig) {
+	if db == nil || pgConfig != *globalPgConfig {
 		// create new connection if not already exists
-		globalPgConfig = pgConfig
-		db = openPostgreSQL(pgConfig)
+		*globalPgConfig = pgConfig
+		db = openPostgreSQL(globalPgConfig)
 	}
 	check(db)
 }
 
 func InitAdminDB() {
 	pgConfig := createPostgreSQLPrimaryAdminConfig()
-	initDB(DB, pgConfig, globalPgConfig)
+	initDB(DB, pgConfig, &globalPgConfig)
 }
 
 func InitUserDB() {
 	pgConfig := createPostgreSQLPrimaryUserConfig()
-	initDB(DB, pgConfig, globalPgConfig)
+	initDB(DB, pgConfig, &globalPgConfig)
 	if utils.CoreCfg.DBReadReplicaEnabled {
 		pgConfig := createPostgreSQLReplicaConfig()
-		initDB(DBReadReplica, pgConfig, globalPgReadReplicaConfig)
+		initDB(DBReadReplica, pgConfig, &globalPgReadReplicaConfig)
 	}
 }
 
@@ -116,7 +116,7 @@ func check(db *gorm.DB) {
 }
 
 // load database config from environment vars using inserted prefix
-func createPostgreSQLPrimaryConfig() *PostgreSQLConfig {
+func createPostgreSQLPrimaryConfig() PostgreSQLConfig {
 	config := PostgreSQLConfig{
 		Host:                   utils.CoreCfg.DBHost,
 		Port:                   utils.CoreCfg.DBPort,
@@ -129,10 +129,10 @@ func createPostgreSQLPrimaryConfig() *PostgreSQLConfig {
 		MaxIdleConnections:     utils.CoreCfg.DBMaxIdleConnections,
 		MaxConnectionLifetimeS: utils.CoreCfg.DBMaxConnectionLifetimeS,
 	}
-	return &config
+	return config
 }
 
-func createPostgreSQLReplicaConfig() *PostgreSQLConfig {
+func createPostgreSQLReplicaConfig() PostgreSQLConfig {
 	config := createPostgreSQLPrimaryConfig()
 	config.User = utils.CoreCfg.DBUser
 	config.Passwd = utils.CoreCfg.DBPassword
@@ -141,14 +141,14 @@ func createPostgreSQLReplicaConfig() *PostgreSQLConfig {
 	return config
 }
 
-func createPostgreSQLPrimaryUserConfig() *PostgreSQLConfig {
+func createPostgreSQLPrimaryUserConfig() PostgreSQLConfig {
 	config := createPostgreSQLPrimaryConfig()
 	config.User = utils.CoreCfg.DBUser
 	config.Passwd = utils.CoreCfg.DBPassword
 	return config
 }
 
-func createPostgreSQLPrimaryAdminConfig() *PostgreSQLConfig {
+func createPostgreSQLPrimaryAdminConfig() PostgreSQLConfig {
 	config := createPostgreSQLPrimaryConfig()
 	config.User = utils.CoreCfg.DBAdminUser
 	config.Passwd = utils.CoreCfg.DBAdminPassword