Github user necouchman commented on a diff in the pull request:

    https://github.com/apache/guacamole-client/pull/274#discussion_r182983804
  
    --- Diff: 
guacamole/src/main/java/org/apache/guacamole/rest/directory/DirectoryResource.java
 ---
    @@ -143,13 +144,14 @@ public DirectoryResource(UserContext userContext, 
Directory<InternalType> direct
     
             // An admin user has access to all objects
             User self = userContext.self();
    -        SystemPermissionSet systemPermissions = 
self.getSystemPermissions();
    +        Permissions effective = self.getEffectivePermissions();
    --- End diff --
    
    Same note as above - any reason not to do
    
        Permissions effective = userContext.self().getEffectivePermissions()
    
    ?


---

Reply via email to