fix(meeting): throw user to login page if the meeting isn't for guests

Author: BreadGenie

frontend/components.d.ts

@@ -54,9 +54,6 @@ declare module "vue" {
 		LucideHand: typeof import("~icons/lucide/hand")["default"];
 		LucideHome: typeof import("~icons/lucide/home")["default"];
 		LucideLoader: typeof import("~icons/lucide/loader")["default"];
-		LucideMessageCircle: typeof import(
-			"~icons/lucide/message-circle",
-		)["default"];
 		LucideMessageSquare: typeof import(
 			"~icons/lucide/message-square",
 		)["default"];

frontend/src/pages/Meeting.vue

@@ -161,7 +161,12 @@
 </template>
 
 <script setup>
-import { Button, Spinner, createDocumentResource } from "frappe-ui";
+import {
+	Button,
+	Spinner,
+	createDocumentResource,
+	frappeRequest,
+} from "frappe-ui";
 import { computed, onMounted, onUnmounted, provide, ref, watch } from "vue";
 import { useRoute, useRouter } from "vue-router";
 
@@ -665,6 +670,29 @@ onMounted(async () => {
 		}
 	}
 
+	// Check meeting access for unauthenticated users
+	if (!session.isLoggedIn) {
+		try {
+			const accessData = await frappeRequest({
+				url: "/api/method/meet.api.meeting.check_meeting_access",
+				params: {
+					meeting_id: meetingId.value,
+				},
+			});
+
+			if (!accessData.allow_guest) {
+				router.push({
+					name: "Login",
+					query: { next: `/${meetingId.value}` },
+				});
+				return;
+			}
+		} catch (error) {
+			console.error("Failed to check meeting access:", error);
+			return;
+		}
+	}
+
 	// Check authentication and handle guest sessions
 	if (!session.isLoggedIn) {
 		const guestId = sessionStorage.getItem("guest_id");

meet/api/meeting.py

@@ -651,3 +651,34 @@ def update_meeting_settings(meeting_id: str, allow_guest: int, meeting_type: str
 		"updated_fields": updated_fields,
 		"message": "Meeting settings updated successfully",
 	}
+
+
+@frappe.whitelist(allow_guest=True)
+@rate_limit(limit=10, seconds=5 * 60)
+def check_meeting_access(meeting_id: str) -> dict:
+	"""
+	Check if a meeting allows guest access without authentication
+
+	Args:
+		meeting_id: The meeting ID to check
+
+	Returns:
+		dict: Access information for the meeting
+	"""
+	try:
+		meeting: SaeMeeting = frappe.get_doc("Sae Meeting", meeting_id)
+
+		return {
+			"success": True,
+			"allow_guest": meeting.allow_guest,
+		}
+	except frappe.DoesNotExistError:
+		return {
+			"success": False,
+			"error": "Meeting not found",
+		}
+	except Exception as e:
+		return {
+			"success": False,
+			"error": str(e),
+		}