diff --git a/src/components/SiderMenu/SiderMenu.js b/src/components/SiderMenu/SiderMenu.js
index e457f2f5b6f12524b26c112302e2e6c0cbe91f67..d0ca78ef68e6fe30d74f64cfb8d63f6f52fc2696 100644
--- a/src/components/SiderMenu/SiderMenu.js
+++ b/src/components/SiderMenu/SiderMenu.js
@@ -98,7 +98,6 @@ export default class SiderMenu extends PureComponent {
 
     return (
       <Sider
-        trigger={null}
         collapsible
         collapsed={collapsed}
         breakpoint="lg"
diff --git a/src/utils/authority.js b/src/utils/authority.js
index 76bd0f7ccde4e07566a4ef0b989f117f95307917..d595df32faf2c36dea985b1f9c3cf60cfdd6d9b5 100644
--- a/src/utils/authority.js
+++ b/src/utils/authority.js
@@ -1,17 +1,14 @@
 // use localStorage to store the authority info, which might be sent from server in actual project.
 export function getAuthority() {
   // return localStorage.getItem('antd-pro-authority') || ['admin', 'user'];
-  let authority = localStorage.getItem('antd-pro-authority');
-  if (authority) {
-    if (authority.includes('[')) {
-      authority = JSON.parse(authority);
-    } else {
-      authority = [JSON.parse(authority)];
-    }
-  } else {
-    authority = ['admin'];
+  const authorityString = localStorage.getItem('antd-pro-authority');
+  let authority;
+  try {
+    authority = JSON.parse(authorityString);
+  } catch (e) {
+    authority = [authorityString];
   }
-  return authority;
+  return authority || ['admin'];
 }
 
 export function setAuthority(authority) {