Author: pauls Date: Fri Oct 13 12:27:01 2017 New Revision: 1812124 URL: http://svn.apache.org/viewvc?rev=1812124&view=rev Log: SLING-7198: Remove duplicated lookup for mapping without subserviceName.
Modified: sling/trunk/bundles/extensions/serviceusermapper/src/main/java/org/apache/sling/serviceusermapping/impl/ServiceUserMapperImpl.java Modified: sling/trunk/bundles/extensions/serviceusermapper/src/main/java/org/apache/sling/serviceusermapping/impl/ServiceUserMapperImpl.java URL: http://svn.apache.org/viewvc/sling/trunk/bundles/extensions/serviceusermapper/src/main/java/org/apache/sling/serviceusermapping/impl/ServiceUserMapperImpl.java?rev=1812124&r1=1812123&r2=1812124&view=diff ============================================================================== --- sling/trunk/bundles/extensions/serviceusermapper/src/main/java/org/apache/sling/serviceusermapping/impl/ServiceUserMapperImpl.java (original) +++ sling/trunk/bundles/extensions/serviceusermapper/src/main/java/org/apache/sling/serviceusermapping/impl/ServiceUserMapperImpl.java Fri Oct 13 12:27:01 2017 @@ -479,14 +479,6 @@ public class ServiceUserMapperImpl imple } } - for (Mapping mapping : this.activeMappings) { - final Iterable<String> principalNames = mapping.mapPrincipals(serviceName, null); - if (principalNames != null) { - log.debug("Got principalNames [{}] from {}/{}", new Object[] {principalNames, serviceName }); - return principalNames; - } - } - // second round without serviceInfo log.debug( "internalGetPrincipalNames: {} active mappings, looking for mapping for {}/<no subServiceName>",