chore: suppress warnings from baserow commands logs (baserow#4502)

* Remove warnings from baserow commands logs

* Address feedback

Author: silvestrid

backend/src/baserow/api/workspaces/users/serializers.py

@@ -46,8 +46,8 @@ def get_email(self, object):
 
     def get_two_factor_auth(self, object):
         try:
-            provider = object.user.two_factor_auth_providers.all()[0]
-        except IndexError:
+            provider = object.user.two_factor_auth_provider
+        except User.two_factor_auth_provider.RelatedObjectDoesNotExist:
             provider = None
 
         return TwoFactorAuthSerializer(provider).data

backend/src/baserow/api/workspaces/users/views.py

@@ -131,7 +131,7 @@ def get(self, request, workspace_id, query_params):
             .select_related("workspace", "user", "user__profile")
             .prefetch_related(
                 Prefetch(
-                    "user__two_factor_auth_providers",
+                    "user__two_factor_auth_provider",
                     queryset=specific_queryset(
                         TwoFactorAuthProviderModel.objects.all()
                     ),

backend/src/baserow/config/asgi.py

@@ -4,7 +4,11 @@
 
 from channels.routing import ProtocolTypeRouter, URLRouter
 
-from baserow.config.helpers import ConcurrencyLimiterASGI, check_lazy_loaded_libraries
+from baserow.config.helpers import (
+    ConcurrencyLimiterASGI,
+    check_lazy_loaded_libraries,
+    log_env_warnings,
+)
 from baserow.core.mcp import get_baserow_mcp_server
 from baserow.core.telemetry.telemetry import setup_logging, setup_telemetry
 from baserow.ws.routers import websocket_router
@@ -22,6 +26,9 @@
 # This runs after Django is fully loaded, so it catches imports from all apps.
 check_lazy_loaded_libraries()
 
+# Finally log any warnings about the environment variables that can help debug issues.
+log_env_warnings()
+
 application = ProtocolTypeRouter(
     {
         "http": ConcurrencyLimiterASGI(

backend/src/baserow/config/helpers.py

@@ -1,4 +1,5 @@
 import asyncio
+import os
 import sys
 
 from django.conf import settings
@@ -75,3 +76,37 @@ def __init__(self, app, max_concurrency: int | None = None):
     async def __call__(self, scope, receive, send):
         async with self.semaphore:
             await self.app(scope, receive, send)
+
+
+def log_env_warnings():
+    from django.conf import settings
+
+    if settings.BASEROW_PUBLIC_URL:
+        if settings.BASEROW_PUBLIC_URL.startswith("http://localhost"):
+            logger.warning(
+                "WARNING: Baserow is configured to use a BASEROW_PUBLIC_URL of "
+                f"{settings.BASEROW_PUBLIC_URL}. If you attempt to access Baserow on "
+                "any other hostname requests to the backend will fail as they will be "
+                "from an unknown host. "
+                "Please set BASEROW_PUBLIC_URL if you will be accessing Baserow "
+                f"from any other URL then {settings.BASEROW_PUBLIC_URL}."
+            )
+    else:
+        if "PUBLIC_BACKEND_URL" not in os.environ:
+            logger.warning(
+                "WARNING: Baserow is configured to use a PUBLIC_BACKEND_URL of "
+                f"{settings.PUBLIC_BACKEND_URL}. If you attempt to access Baserow on any other "
+                "hostname requests to the backend will fail as they will be from an "
+                "unknown host."
+                "Please ensure you set PUBLIC_BACKEND_URL if you will be accessing "
+                f"Baserow from any other URL then {settings.PUBLIC_BACKEND_URL}."
+            )
+        if "PUBLIC_WEB_FRONTEND_URL" not in os.environ:
+            logger.warning(
+                "WARNING: Baserow is configured to use a default PUBLIC_WEB_FRONTEND_URL "
+                f"of {settings.PUBLIC_WEB_FRONTEND_URL}. Emails sent by Baserow will use links "
+                f"pointing to {settings.PUBLIC_WEB_FRONTEND_URL} when telling users how to "
+                "access your server. "
+                "If this is incorrect please ensure you have set PUBLIC_WEB_FRONTEND_URL to "
+                "the URL where users can access your Baserow server."
+            )

backend/src/baserow/config/settings/base.py

@@ -723,40 +723,15 @@ def __setitem__(self, key, value):
     },
 }
 
-BASEROW_PUBLIC_URL = os.getenv("BASEROW_PUBLIC_URL")
+BASEROW_PUBLIC_URL = os.getenv("BASEROW_PUBLIC_URL", "")
 if BASEROW_PUBLIC_URL:
     PUBLIC_BACKEND_URL = BASEROW_PUBLIC_URL
     PUBLIC_WEB_FRONTEND_URL = BASEROW_PUBLIC_URL
-    if BASEROW_PUBLIC_URL == "http://localhost":
-        print(
-            "WARNING: Baserow is configured to use a BASEROW_PUBLIC_URL of "
-            "http://localhost. If you attempt to access Baserow on any other hostname "
-            "requests to the backend will fail as they will be from an unknown host. "
-            "Please set BASEROW_PUBLIC_URL if you will be accessing Baserow "
-            "from any other URL then http://localhost."
-        )
 else:
     PUBLIC_BACKEND_URL = os.getenv("PUBLIC_BACKEND_URL", "http://localhost:8000")
     PUBLIC_WEB_FRONTEND_URL = os.getenv(
         "PUBLIC_WEB_FRONTEND_URL", "http://localhost:3000"
     )
-    if "PUBLIC_BACKEND_URL" not in os.environ:
-        print(
-            "WARNING: Baserow is configured to use a PUBLIC_BACKEND_URL of "
-            "http://localhost:8000. If you attempt to access Baserow on any other "
-            "hostname requests to the backend will fail as they will be from an "
-            "unknown host."
-            "Please ensure you set PUBLIC_BACKEND_URL if you will be accessing "
-            "Baserow from any other URL then http://localhost."
-        )
-    if "PUBLIC_WEB_FRONTEND_URL" not in os.environ:
-        print(
-            "WARNING: Baserow is configured to use a default PUBLIC_WEB_FRONTEND_URL "
-            "of http://localhost:3000. Emails sent by Baserow will use links pointing "
-            "to http://localhost:3000 when telling users how to access your server. If "
-            "this is incorrect please ensure you have set PUBLIC_WEB_FRONTEND_URL to "
-            "the URL where users can access your Baserow server."
-        )
 
 BASEROW_EMBEDDED_SHARE_URL = os.getenv("BASEROW_EMBEDDED_SHARE_URL")
 if not BASEROW_EMBEDDED_SHARE_URL:

backend/src/baserow/config/wsgi.py

@@ -10,7 +10,7 @@
 from django.conf import settings
 from django.core.wsgi import get_wsgi_application
 
-from baserow.config.helpers import check_lazy_loaded_libraries
+from baserow.config.helpers import check_lazy_loaded_libraries, log_env_warnings
 from baserow.core.telemetry.telemetry import setup_logging, setup_telemetry
 
 # The telemetry instrumentation library setup needs to run prior to django's setup.
@@ -28,3 +28,6 @@
 # Check that libraries meant to be lazy-loaded haven't been imported at startup.
 # This runs after Django is fully loaded, so it catches imports from all apps.
 check_lazy_loaded_libraries()
+
+# Finally log any warnings about the environment variables that can help debug issues.
+log_env_warnings()

backend/src/baserow/core/migrations/0111_alter_twofactorauthprovidermodel_user.py

@@ -0,0 +1,28 @@
+import django.db.models.deletion
+from django.conf import settings
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+    dependencies = [
+        ("core", "0110_totpusedcode"),
+        migrations.swappable_dependency(settings.AUTH_USER_MODEL),
+    ]
+
+    operations = [
+        migrations.SeparateDatabaseAndState(
+            state_operations=[
+                migrations.AlterField(
+                    model_name="twofactorauthprovidermodel",
+                    name="user",
+                    field=models.OneToOneField(
+                        help_text="User that setup 2fa with this provider",
+                        on_delete=django.db.models.deletion.CASCADE,
+                        related_name="two_factor_auth_provider",
+                        to=settings.AUTH_USER_MODEL,
+                    ),
+                ),
+            ],
+            database_operations=[],
+        ),
+    ]

backend/src/baserow/core/two_factor_auth/models.py

@@ -21,11 +21,10 @@ class TwoFactorAuthProviderModel(
         related_name="two_factor_auth_providers",
         on_delete=models.CASCADE,
     )
-    user = models.ForeignKey(
+    user = models.OneToOneField(
         "auth.User",
-        unique=True,
         on_delete=models.CASCADE,
-        related_name="two_factor_auth_providers",
+        related_name="two_factor_auth_provider",
         help_text="User that setup 2fa with this provider",
     )
 

backend/src/baserow/manage.py

@@ -13,11 +13,6 @@ def enable_debugger():
         debugger_port = int(os.environ.get("BASEROW_BACKEND_DEBUGGER_PORT", 5678))
         debugpy.listen(("0.0.0.0", debugger_port))  # nosec
         print(f"Debugger attached! Listening on 0.0.0.0:{debugger_port}")
-    else:
-        print(
-            "Debugger disabled. Set the env variable "
-            "BASEROW_BACKEND_DEBUGGER_ENABLED=True to enable it."
-        )
 
 
 def main():