[HADOOP-14743] CompositeGroupsMapping should not swallow exceptions

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.5.0
• Fix Version/s: 2.9.0, 3.0.0-beta1
• Component/s: security
• Target Version/s: 2.9.0, 3.0.0-beta1, 2.8.2, 2.7.5
• Hadoop Flags: Reviewed
• Assignee: Wei-Chiu Chuang
• Reporter: Wei-Chiu Chuang
• Created: 08/Aug/17 00:01
• Updated: 10/Aug/17 17:10
• Resolved: 10/Aug/17 17:10

Description

CompositeGroupsMapping#getGroups

   for (GroupMappingServiceProvider provider : providersList) {
      try {
        groups = provider.getGroups(user);
      } catch (Exception e) {
        //LOG.warn("Exception trying to get groups for user " + user, e);      
      }        
      if (groups != null && ! groups.isEmpty()) {
        groupSet.addAll(groups);
        if (!combined) break;
      }
    }

If anything fails inside the underlying groups mapping service provider, there's no way to tell what went wrong.