You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by bi...@apache.org on 2001/12/08 05:50:41 UTC
cvs commit: jakarta-tomcat/src/share/org/apache/tomcat/modules/aaa AccessInterceptor.java
billbarker 01/12/07 20:50:41
Modified: src/share/org/apache/tomcat/modules/aaa
AccessInterceptor.java
Log:
Fix the position of ;jsessionid on 401 responses.
The original code didn't work if the protected servlet was being called with a query string (it put the ;jsessionid after the query string, so it becomes lost). Now it is stuck to the requestURI where it belongs.
Revision Changes Path
1.20 +4 -3 jakarta-tomcat/src/share/org/apache/tomcat/modules/aaa/AccessInterceptor.java
Index: AccessInterceptor.java
===================================================================
RCS file: /home/cvs/jakarta-tomcat/src/share/org/apache/tomcat/modules/aaa/AccessInterceptor.java,v
retrieving revision 1.19
retrieving revision 1.20
diff -u -r1.19 -r1.20
--- AccessInterceptor.java 2001/10/27 02:12:19 1.19
+++ AccessInterceptor.java 2001/12/08 04:50:41 1.20
@@ -527,9 +527,7 @@
if( debug>0) log( "Username = " + username);
String originalLocation = req.requestURI().toString();
- if (req.queryString().toString() != null
- && !req.queryString().toString().equals(""))
- originalLocation += "?" + req.queryString().toString();
+
//XXX is needed to put the JVM route too?
if (noSession
|| Request.SESSIONID_FROM_URL.equals(req.getSessionIdSource())) {
@@ -538,6 +536,9 @@
originalLocation += id ;
page += id ;
}
+ if (req.queryString().toString() != null
+ && !req.queryString().toString().equals(""))
+ originalLocation += "?" + req.queryString().toString();
session.setAttribute( "tomcat.auth.originalLocation",
originalLocation);
--
To unsubscribe, e-mail: <ma...@jakarta.apache.org>
For additional commands, e-mail: <ma...@jakarta.apache.org>