You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sling.apache.org by "Tobias Bocanegra (JIRA)" <ji...@apache.org> on 2008/02/06 00:29:08 UTC

[jira] Created: (SLING-224) add .res extension to launchpad defaut servlet

add .res extension to launchpad defaut servlet
----------------------------------------------

                 Key: SLING-224
                 URL: https://issues.apache.org/jira/browse/SLING-224
             Project: Sling
          Issue Type: Improvement
          Components: microsling
            Reporter: Tobias Bocanegra


the launchpad serlvet should also handle the .res extension for adaptables to 'stream'.

the reason is, that if a nt:resource node name has no extension, it's hard to request it.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Resolved: (SLING-224) add .res extension to launchpad defaut servlet

Posted by "Felix Meschberger (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/SLING-224?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Felix Meschberger resolved SLING-224.
-------------------------------------

       Resolution: Fixed
    Fix Version/s: 2.0.0
         Assignee: Felix Meschberger

Implemented a slightly different solution by creating a streamer servlet, which is called if the request has no extension or if the request extension is ".res".

Fixed in Rev. 618964.

Please close if this fixes your issue. Thanks.

> add .res extension to launchpad defaut servlet
> ----------------------------------------------
>
>                 Key: SLING-224
>                 URL: https://issues.apache.org/jira/browse/SLING-224
>             Project: Sling
>          Issue Type: Improvement
>          Components: microsling
>            Reporter: Tobias Bocanegra
>            Assignee: Felix Meschberger
>             Fix For: 2.0.0
>
>
> the launchpad serlvet should also handle the .res extension for adaptables to 'stream'.
> the reason is, that if a nt:resource node name has no extension, it's hard to request it.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (SLING-224) add .res extension to launchpad defaut servlet

Posted by "Tobias Bocanegra (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/SLING-224?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12565960#action_12565960 ] 

Tobias Bocanegra commented on SLING-224:
----------------------------------------

suggested patch:

Index: launchpad/launchpad-servlets/src/main/java/org/apache/sling/launchpad/servlets/LaunchpadDefaultServlet.java
===================================================================
--- launchpad/launchpad-servlets/src/main/java/org/apache/sling/launchpad/servlets/LaunchpadDefaultServlet.java (revision 618806)
+++ launchpad/launchpad-servlets/src/main/java/org/apache/sling/launchpad/servlets/LaunchpadDefaultServlet.java (working copy)
@@ -123,7 +123,7 @@
         Servlet s = defaultGetServlet;
         InputStream stream = null;
         final String ext = request.getRequestPathInfo().getExtension();
-        if (ext != null && ext.length() > 0) {
+        if (ext != null && ext.length() > 0 && !ext.equals("res")) {
             // if there is an extension, lookup our getServlets
             s = getServlets.get(ext);
         } else {


> add .res extension to launchpad defaut servlet
> ----------------------------------------------
>
>                 Key: SLING-224
>                 URL: https://issues.apache.org/jira/browse/SLING-224
>             Project: Sling
>          Issue Type: Improvement
>          Components: microsling
>            Reporter: Tobias Bocanegra
>
> the launchpad serlvet should also handle the .res extension for adaptables to 'stream'.
> the reason is, that if a nt:resource node name has no extension, it's hard to request it.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Closed: (SLING-224) add .res extension to launchpad defaut servlet

Posted by "Tobias Bocanegra (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/SLING-224?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Tobias Bocanegra closed SLING-224.
----------------------------------


great. thanks.

> add .res extension to launchpad defaut servlet
> ----------------------------------------------
>
>                 Key: SLING-224
>                 URL: https://issues.apache.org/jira/browse/SLING-224
>             Project: Sling
>          Issue Type: Improvement
>          Components: microsling
>            Reporter: Tobias Bocanegra
>            Assignee: Felix Meschberger
>             Fix For: 2.0.0
>
>
> the launchpad serlvet should also handle the .res extension for adaptables to 'stream'.
> the reason is, that if a nt:resource node name has no extension, it's hard to request it.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.