Hi Felix
>
>
> Marc Speck schrieb:
> > 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.
>
> In fact, your initial message prompted me to finally publicize the
> proposal ;-) So, I didn't want to answer to your request too early.


yeah, I thought so.


>
> Since, the proposal is most probably not going to be implemented within
> a day or two ;-), your proposal makes absolute sense to me. (In fact I
> have implemented some prototype in my workspace to look at it ;-) ).


great!


>
>
> My propotype just extends the AuthenticationInfo class by adding another
> constructor taking the workspace name and a new getWorkspaceName method.

Yep, I did the same.


>
> Would you mind to create a JIRA for this ? Then I will ASAP commit this
> extension.


https://issues.apache.org/jira/browse/SLING-727


>
>
> Thinking of compatibility: It think we can easily turn the
> AuthenticationInfo class into a Map<String, Object> suitable for the
> proposed ResourceProviderFactory.getResourceProvider method for it to be
> used by AuthenticationHandler implementations. So this proposal can
> still be supported even with ResourceResolverFactory implemented.


That's fine with me.

Marc

Reply via email to