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

Emond Papegaaij commented on WICKET-6847:
-----------------------------------------

I'm looking at the change by [~mgrigorov] once more and I think it is 
incorrect. {{RequestPageStore.addPage}} is called from {{Page.onInitialize}}, 
which can be called from {{Page.isPageStateless}} and now 
{{RequestPageStore.addPage}} calls {{Page.isPageStateless}} again. IMHO this is 
bound to give problems. Here's a stack dump from our application (on 9.1.0):

{code}
RequestPageStore.addPage(IPageContext, IManageablePage) line: 62        
DecoratablePageManager(PageManager).touchPage(IManageablePage) line: 67 
DecoratablePageManager.touchPage(IManageablePage) line: 22      
PageAccessSynchronizer$1.touchPage(IManageablePage) line: 150   
ConfigApplicationPage(Page).onInitialize() line: 301    
ConfigApplicationPage(AbstractKeyHubPage).onInitialize() line: 64       
ConfigApplicationPage(AbstractSecureKeyHubPage).onInitialize() line: 84 
ConfigApplicationPage(AbstractKeyHubAdminPage).onInitialize() line: 57  
ConfigApplicationPage.onInitialize() line: 25   
ConfigApplicationPage(Component).fireInitialize() line: 874     
ConfigApplicationPage(MarkupContainer).internalInitialize() line: 1050  
ConfigApplicationPage(Page).isPageStateless() line: 461 
{code}

> async page storing fails with flush before detach without session
> -----------------------------------------------------------------
>
>                 Key: WICKET-6847
>                 URL: https://issues.apache.org/jira/browse/WICKET-6847
>             Project: Wicket
>          Issue Type: Bug
>          Components: wicket
>    Affects Versions: 9.1.0
>            Reporter: Sven Meier
>            Assignee: Martin Tzvetanov Grigorov
>            Priority: Minor
>             Fix For: 9.2.0
>
>
> Since WICKET-6831 the response is flushed before detach.
> RequestPageStore delays storing all of stateful pages until detach; at that 
> moment  AsynchronousPageStore can no longer acquire the required session id.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to