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

Felix Meschberger commented on SLING-2618:
------------------------------------------

Thanks for fixing.

Some comments:

* Can access to the Compiler.isSession field be made method access ? Exposing 
and accessing fields IMHO is bad practice.
* We should add a warning to the metatype descriptor that setting this property 
to false actually creates a deviation from the spec.
                
> the default session usage should be configurable
> ------------------------------------------------
>
>                 Key: SLING-2618
>                 URL: https://issues.apache.org/jira/browse/SLING-2618
>             Project: Sling
>          Issue Type: New Feature
>          Components: Scripting
>            Reporter: Justin Edelson
>            Assignee: Justin Edelson
>             Fix For: Scripting JSP 2.0.26
>
>
> JSP's default to using a session. This must be overridden by doing <%@ page 
> session="false" %> on every JSP page, otherwise a session will be created.
> This behavior should be configurable. Many Sling-based applications don't 
> need sessions and it shouldn't be necessary to add a line to every JSP page 
> (or a common include) just to ensure that something you don't need (a 
> session) isn't created.
> For backwards compatibility, the default should remain true.

--
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