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

Hudson commented on TAP5-2537:
------------------------------

FAILURE: Integrated in tapestry-trunk-freestyle #1561 (See 
[https://builds.apache.org/job/tapestry-trunk-freestyle/1561/])
TAP5-2537: avoid an IllegalStateException when trying to get an 
(jochen.kemnade: rev c4c83a8bfe86a1d1ebe055f83a992933f7c6e5e4)
* 
tapestry-core/src/main/java/org/apache/tapestry5/internal/services/SessionApplicationStatePersistenceStrategy.java
* 
tapestry-core/src/test/java/org/apache/tapestry5/internal/services/SessionApplicationStatePersistenceStrategyTest.java


> ApplicationStateManagerImpl.getIfExists() fails when session is invalid (on 
> Jetty, at least)
> --------------------------------------------------------------------------------------------
>
>                 Key: TAP5-2537
>                 URL: https://issues.apache.org/jira/browse/TAP5-2537
>             Project: Tapestry 5
>          Issue Type: Bug
>          Components: tapestry-core
>    Affects Versions: 5.4
>            Reporter: I D
>            Assignee: Jochen Kemnade
>             Fix For: 5.4.1
>
>
> An IllegalStateException is thrown when calling 
> ApplicationStateManagerImpl.getIfExists() on an invalid session. I would 
> expect the method to catch the exception and return null in this case.
> Stack trace (top only):
> {code:java}
> - 
> org.eclipse.jetty.server.session.AbstractSession.checkValid(AbstractSession.java:106)
>   - 
> org.eclipse.jetty.server.session.AbstractSession.getAttribute(AbstractSession.java:459)
>   - 
> org.apache.tapestry5.internal.services.SessionImpl.getAttribute(SessionImpl.java:52)
>   - 
> org.apache.tapestry5.internal.services.SessionApplicationStatePersistenceStrategy.exists(SessionApplicationStatePersistenceStrategy.java:80)
>   - $ApplicationStatePersistenceStrategy_b43cfdb8299.exists(Unknown Source)
>   - 
> org.apache.tapestry5.internal.services.ApplicationStateManagerImpl$ApplicationStateAdapter.exists(ApplicationStateManagerImpl.java:60)
>   - 
> org.apache.tapestry5.internal.services.ApplicationStateManagerImpl.getIfExists(ApplicationStateManagerImpl.java:140)
> {code}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to