diff --git a/jperm/perm_api.py b/jperm/perm_api.py
index 9e4d014cd..0a6d0b805 100644
--- a/jperm/perm_api.py
+++ b/jperm/perm_api.py
@@ -241,47 +241,39 @@ def get_role_info(role_id, type="all"):
     """
     # 获取role对应的授权规则
     role_obj = PermRole.objects.get(id=role_id)
-    rules_obj = role_obj.perm_rule.all()
+    rule_push_obj = role_obj.perm_rule.all()
     # 获取role 对应的用户 和 用户组
     # 获取role 对应的主机 和主机组
     users_obj = []
     assets_obj = []
     user_groups_obj = []
-    group_users_obj = []
     asset_groups_obj = []
-    group_assets_obj = []
-    for rule in rules_obj:
-        for user in rule.user.all():
+    for push in rule_push_obj:
+        for user in push.user.all():
             users_obj.append(user)
-        for asset in rule.asset.all():
+        for asset in push.asset.all():
             assets_obj.append(asset)
-        for user_group in rule.user_group.all():
+        for user_group in push.user_group.all():
             user_groups_obj.append(user_group)
-            for user in user_group.user_set.all():
-                group_users_obj.append(user)
-        for asset_group in rule.asset_group.all():
+        for asset_group in push.asset_group.all():
             asset_groups_obj.append(asset_group)
-            for asset in asset_group.asset_set.all():
-                group_assets_obj.append(asset)
-
-    calc_users = set(users_obj) | set(group_users_obj)
-    calc_assets = set(assets_obj) | set(group_assets_obj)
 
     if type == "all":
-        return {"rules": rules_obj,
-                "users": list(calc_users),
+        return {"rules": rule_push_obj,
+                "users": users_obj,
                 "user_groups": user_groups_obj,
-                "assets": list(calc_assets),
+                "assets": assets_obj,
                 "asset_groups": asset_groups_obj,
                 }
+
     elif type == "rule":
-        return rules_obj
+        return rule_push_obj
     elif type == "user":
-        return calc_users
+        return users_obj
     elif type == "user_group":
         return user_groups_obj
     elif type == "asset":
-        return calc_assets
+        return assets_obj
     elif type == "asset_group":
         return asset_groups_obj
     else:
@@ -308,7 +300,3 @@ def get_role_push_host(role):
 if __name__ == "__main__":
     print get_role_info(1)
 
-
-
-
-
diff --git a/jperm/views.py b/jperm/views.py
index 44e59b6fe..1c6e0fd37 100644
--- a/jperm/views.py
+++ b/jperm/views.py
@@ -35,7 +35,7 @@ def perm_rule_list(request):
         rules_list = rules_list.filter(Q(name=keyword))
 
     rules_list, p, rules, page_range, current_page, show_first, show_end = pages(rules_list, request)
-        
+
     return my_render('jperm/perm_rule_list.html', locals(), request)
 
 
diff --git a/jumpserver/api.py b/jumpserver/api.py
index 723a4e9e7..77441395e 100644
--- a/jumpserver/api.py
+++ b/jumpserver/api.py
@@ -27,12 +27,12 @@ from django.shortcuts import render_to_response
 from django.core.mail import send_mail
 
 
-def set_log(level):
+def set_log(level, filename='jumpserver.log'):
     """
     return a log file object
     根据提示设置log打印
     """
-    log_file = os.path.join(LOG_DIR, 'jumpserver.log')
+    log_file = os.path.join(LOG_DIR, filename)
     if not os.path.isfile(log_file):
         os.mknod(log_file)
         os.chmod(log_file, 0777)
diff --git a/templates/jperm/perm_rule_detail.html b/templates/jperm/perm_rule_detail.html
index 4a7a835d5..1df2b8ff3 100644
--- a/templates/jperm/perm_rule_detail.html
+++ b/templates/jperm/perm_rule_detail.html
@@ -93,12 +93,15 @@
                                         </tr>
                                     </thead>
                                     <tbody>
-                                    {% for user in users %}
                                         <tr class="gradeX">
+                                            {% for user in users %}
                                             <td class="text-center"> {{ user.name }} </td>
-                                            <td class="text-center"> {{ user | user_which_groups:"group" }} </td>
+                                            {% endfor %}
+                                            {% for group in user_groups %}
+                                            <td class="text-center"> {{ group.name }} </td>
+                                            {% endfor %}
                                         </tr>
-                                    {% endfor %}
+
                                     </tbody>
                                 </table>
                             </div>
@@ -139,12 +142,14 @@
                                         </tr>
                                     </thead>
                                     <tbody>
-                                    {% for asset in assets %}
                                         <tr class="gradeX">
+                                            {% for asset in assets %}
                                             <td class="text-center"> {{ asset.ip }} </td>
-                                            <td class="text-center"> {{ asset | asset_which_groups:"group" }} </td>
+                                            {% endfor %}
+                                            {% for group in asset_groups %}
+                                            <td class="text-center"> {{ group.name }} </td>
+                                            {% endfor %}
                                         </tr>
-                                    {% endfor %}
                                     </tbody>
                                 </table>
                             </div>