You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by kf...@apache.org on 2012/03/07 10:22:09 UTC
svn commit: r1297906 - in /tomcat/tc7.0.x/trunk:
java/org/apache/catalina/authenticator/NonLoginAuthenticator.java
webapps/docs/changelog.xml
Author: kfujino
Date: Wed Mar 7 09:22:09 2012
New Revision: 1297906
URL: http://svn.apache.org/viewvc?rev=1297906&view=rev
Log:
Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=52846.
Modified:
tomcat/tc7.0.x/trunk/java/org/apache/catalina/authenticator/NonLoginAuthenticator.java
tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml
Modified: tomcat/tc7.0.x/trunk/java/org/apache/catalina/authenticator/NonLoginAuthenticator.java
URL: http://svn.apache.org/viewvc/tomcat/tc7.0.x/trunk/java/org/apache/catalina/authenticator/NonLoginAuthenticator.java?rev=1297906&r1=1297905&r2=1297906&view=diff
==============================================================================
--- tomcat/tc7.0.x/trunk/java/org/apache/catalina/authenticator/NonLoginAuthenticator.java (original)
+++ tomcat/tc7.0.x/trunk/java/org/apache/catalina/authenticator/NonLoginAuthenticator.java Wed Mar 7 09:22:09 2012
@@ -116,7 +116,7 @@ public final class NonLoginAuthenticator
LoginConfig config)
throws IOException {
- Principal principal = request.getUserPrincipal();
+ Principal principal = request.getPrincipal();
if (principal != null) {
// excellent... we have already authenticated the client somehow,
// probably from another container that has a login-config
Modified: tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml
URL: http://svn.apache.org/viewvc/tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml?rev=1297906&r1=1297905&r2=1297906&view=diff
==============================================================================
--- tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml (original)
+++ tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml Wed Mar 7 09:22:09 2012
@@ -72,6 +72,11 @@
<bug>52727</bug>, <bug>52729</bug>, <bug>52731</bug> and
<bug>52732</bug>. (markt)
</scode>
+ <fix>
+ <bug>52846</bug>: Make sure NonLoginAuthenticator registers not
+ MemoryUser but GenericPrincipal into a session when UserDatabaseRealm
+ is used. (kfujino)
+ </fix>
</changelog>
</subsection>
<subsection name="Coyote">
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org