Thanks for the feedback, Bertrand. I'm happy to provide this minor patch. However, it seems the new proposal of Felix about ResourceResolverFactory, http://cwiki.apache.org/SLING/add-resourceresolverfactory-service-interface.html, also allows my use case. So should you find agreement on the proposal and implement it in the next weeks, I'm fine with waiting for it.
Marc On Tue, Nov 11, 2008 at 12:10 PM, Bertrand Delacretaz < [EMAIL PROTECTED]> wrote: > Hi Marc, > > On Fri, Nov 7, 2008 at 3:39 PM, Marc Speck <[EMAIL PROTECTED]> wrote: > > > ....We thought about adding the workspaceName to > > AuthenticationInfo(String authType, Credentials credentials, String > > workspaceName).... > > Ok, as long as you keep the existing constructor. > > > ... And then add it in SlingAuthenticator: > > > > ... Session session = getRepository().login( > > - authInfo.getCredentials(), null); > > + authInfo.getCredentials(), > > authInfo.getWorkspaceName());... > > Sounds good to me! > > -Bertrand >
