fix(engine): by default, dump only databases to which user have access

Description

By default, dump all databases to which user have access in case of an empty databases section while running logicalDump

If DLE admin declares a specific list of DBs, then the permissions check won't be applied – unaccessible DBs won't be skipped, so there will be an error at dump runtime. The change affects only "default" path, when the list of DBs is not specified / is empty.

Related issue

#324 (closed)

Checklist

• MR description has been reviewed
• MR changes are functionally tested
• MR does NOT have text changes OR there are text changes and they have been reviewed
• MR does NOT have API/CLI changes OR there are API/CLI changes and they have been reviewed
• MR does NOT have UI changes OR there are UI changes and they have been reviewed

engine/internal/retrieval/engine/postgres/logical/dump.go

@@ -117,7 +117,7 @@ type dumper interface {
 	SetConnectionOptions(context.Context, *Connection) error
 
 	// GetDatabaseListQuery provides the query to get the list of databases for dumping.
-	GetDatabaseListQuery() string
+	GetDatabaseListQuery(username string) string
 }
 
 // Connection provides connection options.
@@ -379,7 +379,7 @@ func (d *DumpJob) getDBList(ctx context.Context) (map[string]DumpDefinition, err
 		return nil, fmt.Errorf("failed to connect to DB: %w", err)
 	}
 
-	rows, err := querier.Query(ctx, d.dumper.GetDatabaseListQuery())
+	rows, err := querier.Query(ctx, d.dumper.GetDatabaseListQuery(d.config.db.Username))
 	if err != nil {
 		return nil, fmt.Errorf("failed to perform query listing databases: %w", err)
 	}