Bug 52443 - Tomcat#defaultRealm shares Realm instance between web applications, resulting in INFO log message from lifecycle
Summary: Tomcat#defaultRealm shares Realm instance between web applications, resulting...
Status: RESOLVED FIXED
Alias: None
Product: Tomcat 7
Classification: Unclassified
Component: Catalina (show other bugs)
Version: unspecified
Hardware: PC Windows XP
: P2 minor (vote)
Target Milestone: ---
Assignee: Tomcat Developers Mailing List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-01-09 16:43 UTC by Konstantin Kolinko
Modified: 2012-01-12 20:51 UTC (History)
0 users



Attachments
2012-01-09_trunk_TestTomcat_twoapps.patch (1.24 KB, patch)
2012-01-09 16:43 UTC, Konstantin Kolinko
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Konstantin Kolinko 2012-01-09 16:43:35 UTC
Created attachment 28129 [details]
2012-01-09_trunk_TestTomcat_twoapps.patch

Inspired by this thread on dev@:
http://tomcat.markmail.org/thread/5qxa7gjsaav4ytcd
"problem using default Realm in new unit tests"

The problem is the following:

1. Tomcat.getDefaultRealm() is effectively a factory method for some Realm instance. It creates this Realm once and caches it as Tomcat#defaultRealm

2. Tomcat.addWebapp() method calls ctx.setRealm(defaultRealm); for every Context that it creates. Thus the Realm instance is shared between web applications.

3. When Context starts it calls start() method on the realm. When the above method was used to create several web applications then during the start of the second and later ones the following message is logged:

09.01.2012 19:19:29 org.apache.catalina.util.LifecycleBase start
INFO: The start() method was called on component [Realm[Simple]] after start() had already been called. The second call will be ignored.

To reproduce:
1) Apply attached patch to org.apache.catalina.startup.TestTomcat of trunk.
2) Run the test.
3) See the above "The start() method was called" message in the logs.


I think there are several ways to resolve this:
a) Do not call start() on the Realm if it is already started, as indicated by Lifecycle.getState()
b) Change Tomcat class to do not share the Realm instance between Contexts. b.1) assign it to the Engine, or
b.2) create a new instance every time.


The a) way may lead to problems when the Context is stopped. It is not clear whether the Realm shall be stopped or not. It it is stopped it will affect another webapp. It may also cause problems with asynchronous start of contexts implemented in 7.0.23+.

The b) way is consistent with what happens when server.xml is parsed.
Comment 1 Konstantin Kolinko 2012-01-09 16:48:29 UTC
One more:
Assigning explicit defaultRealm in Tomcat.addWebapp() prevents us from assign a realm to Engine.  Because of the Realm created in addWebapp() the realm in Engine is never used in the new web application.
Comment 2 Mark Thomas 2012-01-12 20:51:28 UTC
Fixed in trunk and 7.0.x and will be included in 7.0.24 onwards.