- buildSchemaFilter now uses escapeString() to escape single quotes in
schema, only, and ignore filter values.
- Trino driver now wraps this.db with wrapIdentifier() in all queries
(listSchemas, listTables, listTableColumns, buildPaginatedQuery).
- Trino listTableColumns escapes table and schema string literals.
- Added unit tests for SQL injection in buildSchemaFilter and Trino.
- listTables(null) no longer returns []. Uses buildSchemaFilter() to
optionally filter by schema, matching the PostgreSQL driver pattern.
- SSL/TLS connections now work. Certificate files (CA, cert, key) are
read and passed to trino-client. When no certs are provided,
rejectUnauthorized defaults to false (trust server cert).
- Removed server:ssl from Trino disabledFeatures so the SSL toggle
appears in the connection form.
- URL parser now auto-enables ssl for https:// URLs.
- Added integration tests for listTables and SSL (with HTTPS-enabled
Trino testcontainer).
- Added unit tests for SSL config propagation.
- Updated Trino connection docs (EN + ES) with SSL section.
Resolves @rathboma's feedback to show users which information couldn't be displayed due to permissions.
## Changes
- Add permissionWarnings field to TableProperties interface
- Modified PostgreSQL client to collect and return user-friendly permission error messages
- Added "Limited Info" warning indicator in table properties status bar
- Added clickable warning with detailed notification showing what information is restricted
- Updated tests to verify permission warnings are properly returned
## User Experience
- Clear visual indicator when some table information can't be displayed
- Detailed tooltip and notification explaining specific permission limitations
- Relations tab continues to work and shows available information
Co-authored-by: Matthew Rathbone <rathboma@users.noreply.github.com>
- Set _defaultSchema and version properties for test client
- Mock wrapTable method to return properly formatted identifier
- Ensures test client is properly initialized when created with null parameters
Co-authored-by: Matthew Rathbone <rathboma@users.noreply.github.com>
- Modified getTableProperties method to wrap each query in individual error handlers
- Ensures relations are still displayed even when permission-sensitive queries fail
- Added comprehensive unit test to prevent regression
- Fixes issue where Relations tab shows empty for read-only PostgreSQL connections
Resolves#3262
Co-authored-by: rathboma <rathboma@users.noreply.github.com>
Previously, when running a query to a file, internal
erros were being exposed in the UI,
exposing implementation related errors to end users.
This fix ensures that errors are properly handled and
displayed as user-friendly messages instead of raw internal values.