From fd170cdbc01081c12930c490c898ce22486bd929 Mon Sep 17 00:00:00 2001
From: wxg0103 <727495428@qq.com>
Date: Tue, 16 Dec 2025 17:52:27 +0800
Subject: [PATCH] feat: refactor WeCom QR code login handling with improved
 iframe event management

---
 .../user-login/scanCompinents/wecomQrCode.vue | 52 +++++++------------
 1 file changed, 18 insertions(+), 34 deletions(-)

diff --git a/ui/src/views/chat/user-login/scanCompinents/wecomQrCode.vue b/ui/src/views/chat/user-login/scanCompinents/wecomQrCode.vue
index 90635794d..96e23907a 100644
--- a/ui/src/views/chat/user-login/scanCompinents/wecomQrCode.vue
+++ b/ui/src/views/chat/user-login/scanCompinents/wecomQrCode.vue
@@ -3,12 +3,11 @@
 </template>
 
 <script lang="ts" setup>
-import {nextTick, defineProps, onBeforeUnmount} from 'vue'
-import {useRoute, useRouter} from 'vue-router'
-import {getBrowserLang} from '@/locales'
+import { nextTick, defineProps, onBeforeUnmount } from 'vue'
+import { useRoute, useRouter } from 'vue-router'
+import { getBrowserLang } from '@/locales'
 import useStore from '@/stores'
 const WE_COM_ORIGIN = 'https://login.work.weixin.qq.com'
-const LOGIN_SUCCESS_EVENT = 'onLoginSuccess'
 const LOGIN_STATE = 'fit2cloud-wecom-qr'
 const props = defineProps<{
   config: {
@@ -22,10 +21,10 @@ const props = defineProps<{
 
 const router = useRouter()
 const route = useRoute()
-const {chatUser} = useStore()
+const { chatUser } = useStore()
 
 const {
-  params: {accessToken},
+  params: { accessToken },
 } = route as any
 
 let iframe: HTMLIFrameElement | null = null
@@ -53,41 +52,17 @@ function getLang() {
   return lang === 'en-US' ? 'en' : 'zh'
 }
 
-function safeParse(data: unknown) {
-  try {
-    return typeof data === 'string' ? JSON.parse(data) : data
-  } catch {
-    return null
-  }
-}
-
 async function handleLoginSuccess(code: string) {
   await chatUser.wecomCallback(code, accessToken)
 
   router.push({
     name: 'chat',
-    params: {accessToken},
+    params: { accessToken },
     query: route.query,
   })
 }
 
-function handleWindowMessage(event: MessageEvent) {
-  if (event.origin !== WE_COM_ORIGIN) return
-
-  const payload: any = safeParse(event.data)
-  if (!payload?.args) return
-
-  if (payload.args.name === LOGIN_SUCCESS_EVENT) {
-    const code = payload.args?.data?.code
-    if (!code) return
-
-    handleLoginSuccess(code)
-    cleanup()
-  }
-}
-
 function cleanup() {
-  window.removeEventListener('message', handleWindowMessage)
   iframe?.remove()
   iframe = null
 }
@@ -109,9 +84,18 @@ const init = async () => {
     `&state=${LOGIN_STATE}` +
     `&lang=${getLang()}` +
     `&panel_size=small` +
-    `&redirect_type=callback`
-
-  window.addEventListener('message', handleWindowMessage)
+    `&redirect_type=self`
+  iframe.addEventListener('load', (e) => {
+    if (iframe?.contentWindow) {
+      iframe?.contentWindow?.stop()
+      const searchParams = new URLSearchParams(iframe.contentWindow.location.search)
+      const code = searchParams.get('code')
+      if (code) {
+        handleLoginSuccess(code)
+        cleanup()
+      }
+    }
+  })
 }
 
 onBeforeUnmount(cleanup)