[ 
https://issues.apache.org/jira/browse/SLING-2562?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13464559#comment-13464559
 ] 

Felix Meschberger commented on SLING-2562:
------------------------------------------

One comment on the latest patch: The service method has

> if (request.getPathInfo().startsWith("/servletresolver/res/ui")) {

I think the plugin should implement a method:

   private URL getResource(String)

method instead. See http://felix.apache.org/site/providing-resources.html for 
details.
                
> Add a test for servlet resolver as Web Console Tab
> --------------------------------------------------
>
>                 Key: SLING-2562
>                 URL: https://issues.apache.org/jira/browse/SLING-2562
>             Project: Sling
>          Issue Type: Improvement
>          Components: JCR
>            Reporter: Konrad Windszus
>         Attachments: servletresolverplugin.png, SLING-2562-alt2.patch, 
> SLING-2562-alt.patch, SLING-2562-v1.patch
>
>
> Sometimes it is hard to tell which script/servlet answered a specific 
> request. Therefore a tab which provides a test similar to "Configuration 
> Test" in the "Sling Resource Resolver" Tab of the Felix Web Console would be 
> very useful, which would provide the information which script/servlet was 
> used for rendering a response.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to