From 6a19bf4aa786236eb570f95cf4d3eda638f1815c Mon Sep 17 00:00:00 2001 From: Automated Action Date: Sun, 1 Jun 2025 08:25:31 +0000 Subject: [PATCH] Improve CONTENT environment variable handling - Update endpoint to check for both None and empty string values - Add more detailed error message for better user experience - Update documentation to reflect the refined behavior --- app/api/v1/env_vars.py | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/app/api/v1/env_vars.py b/app/api/v1/env_vars.py index a3a6470..a9be8ab 100644 --- a/app/api/v1/env_vars.py +++ b/app/api/v1/env_vars.py @@ -10,7 +10,8 @@ router = APIRouter() "/env/content", response_model=Dict[str, str], summary="Get content from CONTENT environment variable", - description="Returns the value of the CONTENT environment variable" + description="Returns the value of the CONTENT environment variable " + "if set and not empty" ) def get_content_env_var(): """ @@ -20,14 +21,14 @@ def get_content_env_var(): dict: A dictionary containing the CONTENT environment variable value Raises: - HTTPException: If the CONTENT environment variable is not set + HTTPException: If the CONTENT environment variable is not set or is empty """ content_value = os.environ.get("CONTENT") - if content_value is None: + if content_value is None or content_value == "": raise HTTPException( status_code=status.HTTP_404_NOT_FOUND, - detail="CONTENT environment variable is not set" + detail="CONTENT environment variable is not set or is empty" ) return {"content": content_value} \ No newline at end of file