Do not log the database connection string

fixes #1980
This commit is contained in:
Kukks 2020-10-14 09:06:50 +02:00
parent f2870caed2
commit be3f248a5a

View file

@ -112,12 +112,12 @@ namespace BTCPayServer.Hosting
ApplicationDbContextFactory dbContext = null; ApplicationDbContextFactory dbContext = null;
if (!String.IsNullOrEmpty(opts.PostgresConnectionString)) if (!String.IsNullOrEmpty(opts.PostgresConnectionString))
{ {
Logs.Configuration.LogInformation($"Postgres DB used ({opts.PostgresConnectionString})"); Logs.Configuration.LogInformation($"Postgres DB used");
dbContext = new ApplicationDbContextFactory(DatabaseType.Postgres, opts.PostgresConnectionString); dbContext = new ApplicationDbContextFactory(DatabaseType.Postgres, opts.PostgresConnectionString);
} }
else if (!String.IsNullOrEmpty(opts.MySQLConnectionString)) else if (!String.IsNullOrEmpty(opts.MySQLConnectionString))
{ {
Logs.Configuration.LogInformation($"MySQL DB used ({opts.MySQLConnectionString})"); Logs.Configuration.LogInformation($"MySQL DB used");
Logs.Configuration.LogWarning("MySQL is not widely tested and should be considered experimental, we advise you to use postgres instead."); Logs.Configuration.LogWarning("MySQL is not widely tested and should be considered experimental, we advise you to use postgres instead.");
dbContext = new ApplicationDbContextFactory(DatabaseType.MySQL, opts.MySQLConnectionString); dbContext = new ApplicationDbContextFactory(DatabaseType.MySQL, opts.MySQLConnectionString);
} }