You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@superset.apache.org by mi...@apache.org on 2024/02/13 16:02:19 UTC
(superset) 06/16: fix: safer error message in alerts (#27019)
This is an automated email from the ASF dual-hosted git repository.
michaelsmolina pushed a commit to branch 3.1
in repository https://gitbox.apache.org/repos/asf/superset.git
commit c974daa0c7f05bdf8a5e59897c9a2feaaf94bc4e
Author: Beto Dealmeida <ro...@dealmeida.net>
AuthorDate: Wed Feb 7 14:58:51 2024 -0500
fix: safer error message in alerts (#27019)
(cherry picked from commit 686ce33ea5017aad4cca18a6409c00f6b366dcf4)
---
superset/commands/report/alert.py | 9 +++++++--
1 file changed, 7 insertions(+), 2 deletions(-)
diff --git a/superset/commands/report/alert.py b/superset/commands/report/alert.py
index 68013a2c00..fc1be14e02 100644
--- a/superset/commands/report/alert.py
+++ b/superset/commands/report/alert.py
@@ -150,7 +150,7 @@ class AlertCommand(BaseCommand):
rendered_sql, ALERT_SQL_LIMIT
)
- _, username = get_executor(
+ executor, username = get_executor( # pylint: disable=unused-variable
executor_types=app.config["ALERT_REPORTS_EXECUTE_AS"],
model=self._report_schedule,
)
@@ -169,7 +169,12 @@ class AlertCommand(BaseCommand):
logger.warning("A timeout occurred while executing the alert query: %s", ex)
raise AlertQueryTimeout() from ex
except Exception as ex:
- raise AlertQueryError(message=str(ex)) from ex
+ logger.exception("An error occurred when running alert query")
+ # The exception message here can reveal to much information to malicious
+ # users, so we raise a generic message.
+ raise AlertQueryError(
+ message=_("An error occurred when running alert query")
+ ) from ex
def validate(self) -> None:
"""