FIX (databases): Verify DB nil on side of DB instead of interface

RostislavDugin · RostislavDugin · commit 45fc9a7fff48 · 2025-11-09T20:03:22.000+03:00
diff --git a/backend/internal/features/databases/databases/postgresql/model.go b/backend/internal/features/databases/databases/postgresql/model.go
@@ -67,6 +67,10 @@ func (p *PostgresqlDatabase) TestConnection(logger *slog.Logger) error {
 }
 
 func (p *PostgresqlDatabase) HideSensitiveData() {
+	if p == nil {
+		return
+	}
+
 	p.Password = ""
 }
 
diff --git a/backend/internal/features/databases/model.go b/backend/internal/features/databases/model.go
@@ -61,10 +61,7 @@ func (d *Database) TestConnection(logger *slog.Logger) error {
 }
 
 func (d *Database) HideSensitiveData() {
-	specificDB := d.getSpecificDatabase()
-	if specificDB != nil {
-		specificDB.HideSensitiveData()
-	}
+	d.getSpecificDatabase().HideSensitiveData()
 }
 
 func (d *Database) Update(incoming *Database) {

Original file line number	Diff line number	Diff line change
`@@ -67,6 +67,10 @@ func (p PostgresqlDatabase) TestConnection(logger slog.Logger) error {`
`67`	`67`	`}`
`68`	`68`
`69`	`69`	`func (p *PostgresqlDatabase) HideSensitiveData() {`
	`70`	`+ if p == nil {`
	`71`	`+ return`
	`72`	`+ }`
	`73`	`+`
`70`	`74`	`p.Password = ""`
`71`	`75`	`}`
`72`	`76`
Original file line number	Diff line number	Diff line change
`@@ -61,10 +61,7 @@ func (d Database) TestConnection(logger slog.Logger) error {`
`61`	`61`	`}`
`62`	`62`
`63`	`63`	`func (d *Database) HideSensitiveData() {`
`64`		`- specificDB := d.getSpecificDatabase()`
`65`		`- if specificDB != nil {`
`66`		`- specificDB.HideSensitiveData()`
`67`		`- }`
	`64`	`+ d.getSpecificDatabase().HideSensitiveData()`
`68`	`65`	`}`
`69`	`66`
`70`	`67`	`func (d Database) Update(incoming Database) {`