remove default SMTP server, user, and pass

Author: Bo-Onyx

backend/onyx/configs/app_configs.py

@@ -210,10 +210,10 @@
 REQUIRE_EMAIL_VERIFICATION = (
     os.environ.get("REQUIRE_EMAIL_VERIFICATION", "").lower() == "true"
 )
-SMTP_SERVER = os.environ.get("SMTP_SERVER") or "smtp.gmail.com"
+SMTP_SERVER = os.environ.get("SMTP_SERVER") or None
 SMTP_PORT = int(os.environ.get("SMTP_PORT") or "587")
-SMTP_USER = os.environ.get("SMTP_USER", "your-email@gmail.com")
-SMTP_PASS = os.environ.get("SMTP_PASS", "your-gmail-password")
+SMTP_USER = os.environ.get("SMTP_USER") or None
+SMTP_PASS = os.environ.get("SMTP_PASS") or None
 EMAIL_FROM = os.environ.get("EMAIL_FROM") or SMTP_USER
 
 SENDGRID_API_KEY = os.environ.get("SENDGRID_API_KEY") or ""

backend/onyx/server/manage/models.py

@@ -36,10 +36,10 @@
 
 
 class EmailInviteStatus(str, Enum):
-    sent = "sent"
-    not_configured = "not_configured"
-    send_failed = "send_failed"
-    disabled = "disabled"
+    SENT = "SENT"
+    NOT_CONFIGURED = "NOT_CONFIGURED"
+    SEND_FAILED = "SEND_FAILED"
+    DISABLED = "DISABLED"
 
 
 class BulkInviteResponse(BaseModel):

backend/onyx/server/manage/users.py

@@ -431,17 +431,17 @@ def bulk_invite_users(
 
     # send out email invitations only to new users (not already invited or existing)
     if not ENABLE_EMAIL_INVITES:
-        email_invite_status = EmailInviteStatus.disabled
+        email_invite_status = EmailInviteStatus.DISABLED
     elif not EMAIL_CONFIGURED:
-        email_invite_status = EmailInviteStatus.not_configured
+        email_invite_status = EmailInviteStatus.NOT_CONFIGURED
     else:
         try:
             for email in emails_needing_seats:
                 send_user_email_invite(email, current_user, AUTH_TYPE)
-            email_invite_status = EmailInviteStatus.sent
+            email_invite_status = EmailInviteStatus.SENT
         except Exception as e:
             logger.error(f"Error sending email invite to invited users: {e}")
-            email_invite_status = EmailInviteStatus.send_failed
+            email_invite_status = EmailInviteStatus.SEND_FAILED
 
     if not MULTI_TENANT or DEV_MODE:
         return BulkInviteResponse(

backend/tests/unit/onyx/server/manage/test_bulk_invite_limit.py

@@ -48,7 +48,7 @@ def test_trial_tenant_can_invite_within_limit(*_mocks: None) -> None:
     result = bulk_invite_users(emails=emails)
 
     assert result.invited_count == 3
-    assert result.email_invite_status == EmailInviteStatus.disabled
+    assert result.email_invite_status == EmailInviteStatus.DISABLED
 
 
 # --- email_invite_status tests ---
@@ -75,7 +75,7 @@ def test_email_invite_status_disabled(*_mocks: None) -> None:
     """When email invites are disabled, status is disabled."""
     result = bulk_invite_users(emails=["user@example.com"])
 
-    assert result.email_invite_status == EmailInviteStatus.disabled
+    assert result.email_invite_status == EmailInviteStatus.DISABLED
 
 
 @_with_common_patches
@@ -85,7 +85,7 @@ def test_email_invite_status_not_configured(*_mocks: None) -> None:
     """When email invites are enabled but no server is configured, status is not_configured."""
     result = bulk_invite_users(emails=["user@example.com"])
 
-    assert result.email_invite_status == EmailInviteStatus.not_configured
+    assert result.email_invite_status == EmailInviteStatus.NOT_CONFIGURED
 
 
 @_with_common_patches
@@ -97,7 +97,7 @@ def test_email_invite_status_sent(mock_send: MagicMock, *_mocks: None) -> None:
     result = bulk_invite_users(emails=["user@example.com"])
 
     mock_send.assert_called_once()
-    assert result.email_invite_status == EmailInviteStatus.sent
+    assert result.email_invite_status == EmailInviteStatus.SENT
 
 
 @_with_common_patches
@@ -111,5 +111,5 @@ def test_email_invite_status_send_failed(*_mocks: None) -> None:
     """When email sending throws, status is send_failed and invite is still saved."""
     result = bulk_invite_users(emails=["user@example.com"])
 
-    assert result.email_invite_status == EmailInviteStatus.send_failed
+    assert result.email_invite_status == EmailInviteStatus.SEND_FAILED
     assert result.invited_count == 1

web/src/app/admin/users/page.tsx

@@ -273,11 +273,11 @@ function AddUserButton() {
     );
     setBulkAddUsersModal(false);
     toast.success("Users invited!");
-    if (emailInviteStatus === "not_configured") {
+    if (emailInviteStatus === "NOT_CONFIGURED") {
       toast.warning(
         "No email server is configured. Share the signup link with invited users manually."
       );
-    } else if (emailInviteStatus === "send_failed") {
+    } else if (emailInviteStatus === "SEND_FAILED") {
       toast.warning(
         "Email sending failed. Check your SMTP/SendGrid configuration."
       );