perf: 异地登陆去掉本地判断

apps/acls/serializers/login_acl.py

@@ -2,6 +2,7 @@ from django.utils.translation import ugettext as _
 from rest_framework import serializers
 from common.drf.serializers import BulkModelSerializer
 from common.drf.serializers import MethodSerializer
+from jumpserver.utils import has_valid_xpack_license
 from ..models import LoginACL
 from .rules import RuleSerializer
 
@@ -40,12 +41,11 @@ class LoginACLSerializer(BulkModelSerializer):
         self.set_action_choices()
 
     def set_action_choices(self):
-        from xpack.plugins.license.models import License
         action = self.fields.get('action')
         if not action:
             return
         choices = action._choices
-        if not License.has_valid_license():
+        if not has_valid_xpack_license():
             choices.pop(LoginACL.ActionChoices.confirm, None)
         action._choices = choices
 

apps/authentication/utils.py

@@ -59,6 +59,10 @@ def check_different_city_login(user, request):
     else:
         city = get_ip_city(ip) or DEFAULT_CITY
 
-    last_user_login = UserLoginLog.objects.filter(username=user.username, status=True).first()
+    city_white = ['LAN', ]
-    if last_user_login and last_user_login.city != city:
+    if city not in city_white:
-        DifferentCityLoginMessage(user, ip, city).publish_async()
+        last_user_login = UserLoginLog.objects.exclude(city__in=city_white) \
+            .filter(username=user.username, status=True).first()
+
+        if last_user_login and last_user_login.city != city:
+            DifferentCityLoginMessage(user, ip, city).publish_async()