Repository navigation
Sanitise trino errors - #430
Open
Bohreromir wants to merge 2 commits into
Open
Bohreromir wants to merge 2 commits into
Bohreromir wants to merge 2 commits into
Conversation
Trino error messages and raw upstream response bodies were passed to the browser verbatim. Errors are now classified on the server: - USER_ERROR and INSUFFICIENT_RESOURCES keep Trino's message, as they describe the user's own query. - INTERNAL_ERROR, EXTERNAL and unknown types only expose the error name; the message is logged. - HTTP and network failures are reported as unavailable; the response body is only logged. Query snapshots carry the classification as a structured error that the client translates. The catalog and completion metadata endpoints return a generic message with the request ID for anything but query errors.
Add mock Trino markers for an internal error and an HTTP 502 while polling, and check that the UI shows the generic text without the internal details.
Coverage Report
File Coverage
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
This classifies Trino errors and passes some to the user and keeping potentially sensitive ones to the server while informing the user generic
closes #304
Docs
Release note
Checklist
Author
Reviewer
Acceptance