[ 
https://issues.apache.org/jira/browse/WICKET-5557?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Martin Grigorov updated WICKET-5557:
------------------------------------

    Summary: Don't cache the result of 
AuthenticatedWebApplication#getWebSessionClass()  (was: getWebSessionClass() in 
AuthenticatedWebApplication must be constant)

> Don't cache the result of AuthenticatedWebApplication#getWebSessionClass()
> --------------------------------------------------------------------------
>
>                 Key: WICKET-5557
>                 URL: https://issues.apache.org/jira/browse/WICKET-5557
>             Project: Wicket
>          Issue Type: Bug
>          Components: wicket-auth-roles
>    Affects Versions: 6.14.0
>            Reporter: William R. Speirs
>            Priority: Minor
>         Attachments: wicket-5557v1.patch
>
>
> Currently the AuthenticatedWebApplication constructor obtains and holds a 
> reference to the AbstractAuthenticatedWebSession class. This makes it 
> impossible for classes extending AuthenticatedWebApplication to use a class 
> based upon something passed into their constructors. The only option is to 
> provide a "static"/"constant" class from the getWebSessionClass() method.
> I don't see any advantage to holding this reference. Instead I propose that 
> the class is changed to simply calling getWebSessionClass() each time the 
> class is needed.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to