mirror of
https://github.com/zulip/zulip.git
synced 2025-11-02 13:03:29 +00:00
error_notify: Fix type narrowing of settings.ERROR_BOT.
Signed-off-by: Zixuan James Li <p359101898@gmail.com>
This commit is contained in:
committed by
Tim Abbott
parent
c5075be9aa
commit
0ef5eee3f0
@@ -43,7 +43,7 @@ def deployment_repr(report: Dict[str, Any]) -> str:
|
||||
def notify_browser_error(report: Dict[str, Any]) -> None:
|
||||
report = defaultdict(lambda: None, report)
|
||||
if settings.ERROR_BOT:
|
||||
zulip_browser_error(report)
|
||||
zulip_browser_error(report, settings.ERROR_BOT)
|
||||
email_browser_error(report)
|
||||
|
||||
|
||||
@@ -80,7 +80,7 @@ Deployed version: {version}
|
||||
mail_admins(email_subject, body)
|
||||
|
||||
|
||||
def zulip_browser_error(report: Dict[str, Any]) -> None:
|
||||
def zulip_browser_error(report: Dict[str, Any], error_bot_email: str) -> None:
|
||||
email_subject = "JS error: {user_email}".format(**report)
|
||||
|
||||
user_info = user_info_str(report)
|
||||
@@ -89,7 +89,7 @@ def zulip_browser_error(report: Dict[str, Any]) -> None:
|
||||
body += "Message: {message}\n".format(**report)
|
||||
|
||||
error_bot_realm = get_realm(settings.STAFF_SUBDOMAIN)
|
||||
error_bot = get_system_bot(settings.ERROR_BOT, error_bot_realm.id)
|
||||
error_bot = get_system_bot(error_bot_email, error_bot_realm.id)
|
||||
errors_stream = get_stream("errors", error_bot_realm)
|
||||
|
||||
internal_send_stream_message(
|
||||
@@ -104,10 +104,10 @@ def notify_server_error(report: Dict[str, Any]) -> None:
|
||||
report = defaultdict(lambda: None, report)
|
||||
email_server_error(report)
|
||||
if settings.ERROR_BOT:
|
||||
zulip_server_error(report)
|
||||
zulip_server_error(report, settings.ERROR_BOT)
|
||||
|
||||
|
||||
def zulip_server_error(report: Dict[str, Any]) -> None:
|
||||
def zulip_server_error(report: Dict[str, Any], error_bot_email: str) -> None:
|
||||
email_subject = "{node}: {message}".format(**report)
|
||||
|
||||
logger_str = logger_repr(report)
|
||||
@@ -143,7 +143,7 @@ Error generated by {user_info}
|
||||
{request_repr}"""
|
||||
|
||||
error_bot_realm = get_realm(settings.STAFF_SUBDOMAIN)
|
||||
error_bot = get_system_bot(settings.ERROR_BOT, error_bot_realm.id)
|
||||
error_bot = get_system_bot(error_bot_email, error_bot_realm.id)
|
||||
errors_stream = get_stream("errors", error_bot_realm)
|
||||
|
||||
internal_send_stream_message(
|
||||
|
||||
Reference in New Issue
Block a user