You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by co...@apache.org on 2001/07/17 06:06:29 UTC
cvs commit: jakarta-tomcat/src/share/org/apache/tomcat/modules/server Ajp13.java
costin 01/07/16 21:06:29
Modified: src/share/org/apache/tomcat/modules/server Ajp13.java
Log:
Quick fix for 2629 ( userPrincipal not set when used with Ajp13 ).
Full solution ( i.e. real integration of user authentication between
tomcat and apache ) is unlikely to happen in 3.3, but may be provided
by an add-on module ( like j-t-c and ajp14 )
Revision Changes Path
1.20 +4 -0 jakarta-tomcat/src/share/org/apache/tomcat/modules/server/Ajp13.java
Index: Ajp13.java
===================================================================
RCS file: /home/cvs/jakarta-tomcat/src/share/org/apache/tomcat/modules/server/Ajp13.java,v
retrieving revision 1.19
retrieving revision 1.20
diff -u -r1.19 -r1.20
--- Ajp13.java 2001/06/24 22:42:20 1.19
+++ Ajp13.java 2001/07/17 04:06:29 1.20
@@ -70,6 +70,7 @@
import org.apache.tomcat.util.*;
import org.apache.tomcat.util.http.MimeHeaders;
import org.apache.tomcat.util.buf.MessageBytes;
+import org.apache.tomcat.util.aaa.SimplePrincipal;
import org.apache.tomcat.util.http.HttpMessages;
/* Frozen, bug fixes only: all active development goes in
@@ -327,6 +328,9 @@
case SC_A_REMOTE_USER :
req.setRemoteUser( msg.getString());
+ // XXX recycle ?
+ // Note that roles are not integrated with apache
+ req.setUserPrincipal( new SimplePrincipal( msg.getString()));
break;
case SC_A_AUTH_TYPE :