remove dsn from error message to prevent password leaks

mmichaelb · mmichaelb · commit 5bc73e5f9e4c · 2025-08-25T14:37:13.000+02:00
diff --git a/redshift/config.go b/redshift/config.go
@@ -121,7 +121,7 @@ func (c *Client) Connect() (*DBConnection, error) {
 	if !found || conn.Ping() != nil {
 		db, err := sql.Open(driverName, dsn)
 		if err != nil {
-			return nil, fmt.Errorf("error creating Redshift driver instance (driver: %q, dsn: %q): %w", driverName, dsn, err)
+			return nil, fmt.Errorf("error creating Redshift driver instance (driver: %q): %w", driverName, err)
 		}
 
 		// We don't want to retain connection

Original file line number	Diff line number	Diff line change
`@@ -121,7 +121,7 @@ func (c Client) Connect() (DBConnection, error) {`
`121`	`121`	`if !found \|\| conn.Ping() != nil {`
`122`	`122`	`db, err := sql.Open(driverName, dsn)`
`123`	`123`	`if err != nil {`
`124`		`- return nil, fmt.Errorf("error creating Redshift driver instance (driver: %q, dsn: %q): %w", driverName, dsn, err)`
	`124`	`+ return nil, fmt.Errorf("error creating Redshift driver instance (driver: %q): %w", driverName, err)`
`125`	`125`	`}`
`126`	`126`
`127`	`127`	`// We don't want to retain connection`