mirror of
https://github.com/apache/archiva.git
synced 2025-02-07 18:49:48 +00:00
fix possible NPE when restarting a tomcat with serialized session
git-svn-id: https://svn.apache.org/repos/asf/archiva/trunk@1443484 13f79535-47bb-0310-9956-ffa450edef68
This commit is contained in:
parent
9c07475c30
commit
9ff7c33332
@ -50,11 +50,18 @@ public class TemporaryGroupIndexSessionCleaner
|
||||
public void sessionCreated( HttpSessionEvent httpSessionEvent )
|
||||
{
|
||||
// ensure the map is here to avoid NPE
|
||||
httpSessionEvent.getSession().setAttribute( TEMPORARY_INDEX_SESSION_KEY,
|
||||
new HashMap<String, TemporaryGroupIndex>() );
|
||||
WebApplicationContext webApplicationContext = WebApplicationContextUtils.getRequiredWebApplicationContext(
|
||||
httpSessionEvent.getSession().getServletContext() );
|
||||
indexMerger = webApplicationContext.getBean( IndexMerger.class );
|
||||
if ( httpSessionEvent.getSession().getAttribute( TEMPORARY_INDEX_SESSION_KEY ) == null )
|
||||
{
|
||||
httpSessionEvent.getSession().setAttribute( TEMPORARY_INDEX_SESSION_KEY,
|
||||
new HashMap<String, TemporaryGroupIndex>() );
|
||||
}
|
||||
|
||||
if ( indexMerger == null )
|
||||
{
|
||||
WebApplicationContext webApplicationContext = WebApplicationContextUtils.getRequiredWebApplicationContext(
|
||||
httpSessionEvent.getSession().getServletContext() );
|
||||
indexMerger = webApplicationContext.getBean( IndexMerger.class );
|
||||
}
|
||||
}
|
||||
|
||||
public void sessionDestroyed( HttpSessionEvent httpSessionEvent )
|
||||
@ -67,8 +74,19 @@ public void sessionDestroyed( HttpSessionEvent httpSessionEvent )
|
||||
{
|
||||
log.info( "cleanup temporaryGroupIndex {} directory {}", temporaryGroupIndex.getIndexId(),
|
||||
temporaryGroupIndex.getDirectory().getAbsolutePath() );
|
||||
indexMerger.cleanTemporaryGroupIndex( temporaryGroupIndex );
|
||||
getIndexMerger( httpSessionEvent ).cleanTemporaryGroupIndex( temporaryGroupIndex );
|
||||
}
|
||||
}
|
||||
|
||||
private IndexMerger getIndexMerger( HttpSessionEvent httpSessionEvent )
|
||||
{
|
||||
if ( indexMerger == null )
|
||||
{
|
||||
WebApplicationContext webApplicationContext = WebApplicationContextUtils.getRequiredWebApplicationContext(
|
||||
httpSessionEvent.getSession().getServletContext() );
|
||||
indexMerger = webApplicationContext.getBean( IndexMerger.class );
|
||||
}
|
||||
return indexMerger;
|
||||
}
|
||||
}
|
||||
|
||||
|
Loading…
x
Reference in New Issue
Block a user