From 7368605975be59962216082b2dd50f6a089a1298 Mon Sep 17 00:00:00 2001 From: Masen Furer Date: Fri, 22 Nov 2024 11:38:42 -0800 Subject: [PATCH] Avoid `set_log_level` foot gun --- reflex/utils/console.py | 15 +++++++++++++++ 1 file changed, 15 insertions(+) diff --git a/reflex/utils/console.py b/reflex/utils/console.py index 04e590910..b3ba7163d 100644 --- a/reflex/utils/console.py +++ b/reflex/utils/console.py @@ -26,7 +26,22 @@ def set_log_level(log_level: LogLevel): Args: log_level: The log level to set. + + Raises: + ValueError: If the log level is invalid. """ + if not isinstance(log_level, LogLevel): + deprecate( + feature_name="Passing a string to set_log_level", + reason="use reflex.constants.LogLevel enum instead", + deprecation_version="0.6.6", + removal_version="0.7.0", + ) + try: + log_level = getattr(LogLevel, log_level.upper()) + except AttributeError as ae: + raise ValueError(f"Invalid log level: {log_level}") from ae + global _LOG_LEVEL _LOG_LEVEL = log_level