fix: 优化登陆页面特征信息 (#7027)

Refs #7000

Author: ssongliu

backend/app/api/v1/auth.go

@@ -2,6 +2,7 @@ package v1
 
 import (
 	"encoding/base64"
+	"net/http"
 
 	"github.com/1Panel-dev/1Panel/backend/app/api/v1/helper"
 	"github.com/1Panel-dev/1Panel/backend/app/dto"
@@ -120,12 +121,13 @@ func (b *BaseApi) CheckIsSafety(c *gin.Context) {
 		return
 	}
 	if status == "disable" && len(code) != 0 {
-		helper.ErrorWithDetail(c, constant.CodeErrNotFound, constant.ErrTypeInternalServer, err)
+		helper.ErrResponse(c, http.StatusNotFound)
 		return
 	}
 	if status == "unpass" {
-		if middleware.LoadErrCode("err-entrance") != 200 {
-			helper.ErrResponse(c, middleware.LoadErrCode("err-entrance"))
+		code := middleware.LoadErrCode()
+		if code != 200 {
+			helper.ErrResponse(c, code)
 			return
 		}
 		helper.ErrorWithDetail(c, constant.CodeErrEntrance, constant.ErrTypeInternalServer, nil)

backend/middleware/bind_domain.go

@@ -29,8 +29,9 @@ func BindDomain() gin.HandlerFunc {
 		}
 
 		if domains != status.Value {
-			if LoadErrCode("err-domain") != 200 {
-				helper.ErrResponse(c, LoadErrCode("err-domain"))
+			code := LoadErrCode()
+			if code != 200 {
+				helper.ErrResponse(c, code)
 				return
 			}
 			helper.ErrorWithDetail(c, constant.CodeErrDomain, constant.ErrTypeInternalServer, errors.New("domain not allowed"))

backend/middleware/helper.go

@@ -6,7 +6,7 @@ import (
 	"github.com/1Panel-dev/1Panel/backend/app/repo"
 )
 
-func LoadErrCode(errInfo string) int {
+func LoadErrCode() int {
 	settingRepo := repo.NewISettingRepo()
 	codeVal, err := settingRepo.Get(settingRepo.WithByKey("NoAuthSetting"))
 	if err != nil {

backend/middleware/ip_limit.go

@@ -35,8 +35,9 @@ func WhiteAllow() gin.HandlerFunc {
 				return
 			}
 		}
-		if LoadErrCode("err-ip") != 200 {
-			helper.ErrResponse(c, LoadErrCode("err-ip"))
+		code := LoadErrCode()
+		if code != 200 {
+			helper.ErrResponse(c, code)
 			return
 		}
 		helper.ErrorWithDetail(c, constant.CodeErrIP, constant.ErrTypeInternalServer, errors.New("IP address not allowed"))

frontend/src/api/index.ts

@@ -132,12 +132,13 @@ class RequestHttp {
                             );
                             return;
                         default:
+                            globalStore.isLogin = false;
                             globalStore.errStatus = 'code-' + response.status;
                             router.push({
                                 name: 'entrance',
                                 params: { code: globalStore.entrance },
                             });
-                            return;
+                            return Promise.reject(error);
                     }
                 }
                 if (!window.navigator.onLine) router.replace({ path: '/500' });

frontend/src/components/error-message/error_code.vue

@@ -33,9 +33,3 @@ const loadErrInfo = () => {
     }
 };
 </script>
-
-<style scoped lang="scss">
-.container {
-    margin-left: 40%;
-}
-</style>

frontend/src/views/login/entrance/index.vue

@@ -1,17 +1,19 @@
 <template>
     <div>
-        <div v-if="!loading">
-            <div class="login-background" v-if="errStatus === ''">
-                <div class="login-wrapper">
-                    <div :class="screenWidth > 1110 ? 'left inline-block' : ''">
-                        <div class="login-title">
-                            <span>{{ globalStore.themeConfig.title || $t('setting.description') }}</span>
+        <div v-if="init">
+            <div v-if="errStatus === ''">
+                <div class="login-background">
+                    <div class="login-wrapper">
+                        <div :class="screenWidth > 1110 ? 'left inline-block' : ''">
+                            <div class="login-title">
+                                <span>{{ globalStore.themeConfig.title || $t('setting.description') }}</span>
+                            </div>
+                            <img src="@/assets/images/1panel-login.png" alt="" v-if="screenWidth > 1110" />
                         </div>
-                        <img src="@/assets/images/1panel-login.png" alt="" v-if="screenWidth > 1110" />
-                    </div>
-                    <div :class="screenWidth > 1110 ? 'right inline-block' : ''">
-                        <div class="login-container">
-                            <LoginForm ref="loginRef"></LoginForm>
+                        <div :class="screenWidth > 1110 ? 'right inline-block' : ''">
+                            <div class="login-container">
+                                <LoginForm ref="loginRef"></LoginForm>
+                            </div>
                         </div>
                     </div>
                 </div>
@@ -46,45 +48,40 @@ import ErrIP from '@/components/error-message/err_ip.vue';
 import ErrCode from '@/components/error-message/error_code.vue';
 import ErrDomain from '@/components/error-message/err_domain.vue';
 import ErrFound from '@/components/error-message/404.vue';
-import { ref, onMounted, watch } from 'vue';
+import { ref, onMounted } from 'vue';
 import { GlobalStore } from '@/store';
 import { getXpackSettingForTheme } from '@/utils/xpack';
 const globalStore = GlobalStore();
 
 const screenWidth = ref(null);
-const errStatus = ref('');
-const loading = ref();
+const errStatus = ref('x');
+const init = ref(false);
 
 const mySafetyCode = defineProps({
     code: {
         type: String,
         default: '',
     },
 });
-watch(
-    () => globalStore.errStatus,
-    (newVal) => {
-        if (newVal?.startsWith('err-') || newVal?.startsWith('code-')) {
-            errStatus.value = newVal;
-        }
-    },
-);
+
 const getStatus = async () => {
-    let info = globalStore.errStatus;
-    if (info?.startsWith('err-') || info?.startsWith('code-')) {
-        errStatus.value = info;
-        return;
-    }
     let code = mySafetyCode.code;
     globalStore.entrance = code;
-    loading.value = true;
     await checkIsSafety(code)
         .then(() => {
-            loading.value = false;
+            let info = globalStore.errStatus;
+            if (info?.startsWith('err-') || info?.startsWith('code-')) {
+                errStatus.value = info;
+                init.value = true;
+                return;
+            }
+            errStatus.value = '';
+            init.value = true;
             getXpackSettingForTheme();
         })
-        .catch(() => {
-            loading.value = false;
+        .catch((err) => {
+            errStatus.value = 'code-' + err.status;
+            init.value = true;
         });
 };