svn commit: r614171 - /tomcat/tc6.0.x/trunk/STATUS.txt

[remm]

Author: remm
Date: Tue Jan 22 02:59:17 2008
New Revision: 614171

URL: http://svn.apache.org/viewvc?rev=614171&view=rev
Log:
- Votes.

Modified:
tomcat/tc6.0.x/trunk/STATUS.txt

Modified: tomcat/tc6.0.x/trunk/STATUS.txt
URL: 
http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/STATUS.txt?rev=614171&r1=614170&r2=614171&view=diff
==
--- tomcat/tc6.0.x/trunk/STATUS.txt (original)
+++ tomcat/tc6.0.x/trunk/STATUS.txt Tue Jan 22 02:59:17 2008
@@ -35,7 +35,8 @@
   remote JMX access.
   http://svn.apache.org/viewvc?rev=612971&view=rev
   +1: pero, rjung
-  -1:
+  -1: remm: I believe it would need specific support for certain managers. 
Interested users
+could use their own extended manager instead.
 
 * Fix ManagerServlet.expireSessions throws Exceptions as iterate longer 
session lists 
   at production servers.
@@ -52,18 +53,18 @@
   Add support for remaining truststore system property
   Not essential for 6.0.16.
   http://svn.apache.org/viewvc?rev=613266&view=rev
-  +1: markt, pero
+  +1: markt, pero, remm
   -1:
-  
+
 * Update JNDI docs to use server.xml less and  more
   http://svn.apache.org/viewvc?rev=613689&view=rev
   Not essential for 6.0.16.
-  +1: markt, pero
+  +1: markt, pero, remm
   -1: 
 
 * Fix http://issues.apache.org/bugzilla/show_bug.cgi?id=44268
   Log a warning when a duplicate listener is ignored.
   http://svn.apache.org/viewvc?rev=614012&view=rev
   Not essential for 6.0.16
-  +1: markt
+  +1: markt, remm
   -1: 



-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

svn commit: r614189 - /tomcat/tc6.0.x/trunk/STATUS.txt

[pero]

Author: pero
Date: Tue Jan 22 04:22:33 2008
New Revision: 614189

URL: http://svn.apache.org/viewvc?rev=614189&view=rev
Log:
Cast my vote

Modified:
tomcat/tc6.0.x/trunk/STATUS.txt

Modified: tomcat/tc6.0.x/trunk/STATUS.txt
URL: 
http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/STATUS.txt?rev=614189&r1=614188&r2=614189&view=diff
==
--- tomcat/tc6.0.x/trunk/STATUS.txt (original)
+++ tomcat/tc6.0.x/trunk/STATUS.txt Tue Jan 22 04:22:33 2008
@@ -66,5 +66,5 @@
   Log a warning when a duplicate listener is ignored.
   http://svn.apache.org/viewvc?rev=614012&view=rev
   Not essential for 6.0.16
-  +1: markt, remm
+  +1: markt, remm, pero
   -1: 



-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

svn commit: r614213 - /tomcat/tc6.0.x/trunk/STATUS.txt

[fhanik]

Author: fhanik
Date: Tue Jan 22 06:59:05 2008
New Revision: 614213

URL: http://svn.apache.org/viewvc?rev=614213&view=rev
Log:
timeout fix and vote

Modified:
tomcat/tc6.0.x/trunk/STATUS.txt

Modified: tomcat/tc6.0.x/trunk/STATUS.txt
URL: 
http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/STATUS.txt?rev=614213&r1=614212&r2=614213&view=diff
==
--- tomcat/tc6.0.x/trunk/STATUS.txt (original)
+++ tomcat/tc6.0.x/trunk/STATUS.txt Tue Jan 22 06:59:05 2008
@@ -34,7 +34,7 @@
 * Add ManagerBase session getLastAccessedTimestamp and getCreationTimestamp 
for better
   remote JMX access.
   http://svn.apache.org/viewvc?rev=612971&view=rev
-  +1: pero, rjung
+  +1: pero, rjung, fhanik
   -1: remm: I believe it would need specific support for certain managers. 
Interested users
 could use their own extended manager instead.
 
@@ -68,3 +68,9 @@
   Not essential for 6.0.16
   +1: markt, remm, pero
   -1: 
+
+* Fix regression to removal of connection specific comet timeout
+  http://people.apache.org/~fhanik/patches/comet-timeout.patch
+  Would like for 6.0.16, trivial fix
+  +1: fhanik
+  -1:



-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

svn commit: r614249 - /tomcat/trunk/java/org/apache/coyote/http11/Http11NioProcessor.java

[fhanik]

Author: fhanik
Date: Tue Jan 22 08:40:01 2008
New Revision: 614249

URL: http://svn.apache.org/viewvc?rev=614249&view=rev
Log:
bring back comet timeout

Modified:
tomcat/trunk/java/org/apache/coyote/http11/Http11NioProcessor.java

Modified: tomcat/trunk/java/org/apache/coyote/http11/Http11NioProcessor.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/coyote/http11/Http11NioProcessor.java?rev=614249&r1=614248&r2=614249&view=diff
==
--- tomcat/trunk/java/org/apache/coyote/http11/Http11NioProcessor.java 
(original)
+++ tomcat/trunk/java/org/apache/coyote/http11/Http11NioProcessor.java Tue Jan 
22 08:40:01 2008
@@ -889,6 +889,10 @@
 NioEndpoint.KeyAttachment attach = 
(NioEndpoint.KeyAttachment) key.attachment();
 if (attach != null)  {
 attach.setComet(comet);
+if (comet) {
+Integer comettimeout = (Integer) 
request.getAttribute("org.apache.tomcat.comet.timeout");
+if (comettimeout != null) 
attach.setTimeout(comettimeout.longValue());
+}
 }
 }
 } catch (InterruptedIOException e) {



-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

svn commit: r614268 - /tomcat/trunk/java/org/apache/coyote/http11/Http11NioProcessor.java

[fhanik]

Author: fhanik
Date: Tue Jan 22 09:33:32 2008
New Revision: 614268

URL: http://svn.apache.org/viewvc?rev=614268&view=rev
Log:
add the ability to set timeout during any EVENT, not just begin

Modified:
tomcat/trunk/java/org/apache/coyote/http11/Http11NioProcessor.java

Modified: tomcat/trunk/java/org/apache/coyote/http11/Http11NioProcessor.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/coyote/http11/Http11NioProcessor.java?rev=614268&r1=614267&r2=614268&view=diff
==
--- tomcat/trunk/java/org/apache/coyote/http11/Http11NioProcessor.java 
(original)
+++ tomcat/trunk/java/org/apache/coyote/http11/Http11NioProcessor.java Tue Jan 
22 09:33:32 2008
@@ -751,7 +751,10 @@
 NioEndpoint.KeyAttachment attach = 
(NioEndpoint.KeyAttachment)socket.getAttachment(false);
 if (attach != null) {
 attach.setComet(comet);
-if (!comet) {
+if (comet) {
+Integer comettimeout = (Integer) 
request.getAttribute("org.apache.tomcat.comet.timeout");
+if (comettimeout != null) 
attach.setTimeout(comettimeout.longValue());
+} else {
 //reset the timeout
 
attach.setTimeout(endpoint.getSocketProperties().getSoTimeout());
 }



-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

isapi_redirector.vbs Tomcat Connector IIS 7 Compatibility

[Stusynski, Dan]

Greetings,
 
I searched around and couldn't find anything related so I thought I'd
pose the question.
 
I am curious if there will be any movement, or if there already has
been, to update the isapi_redirector.vbs IIS ISAPI configuration script
for IIS 7.0 compatibility? As it stands, 1.2.26's version requires the
enabling of the optional IIS7 role service "IIS 6 Metabase
Compatiblilty" to work. After enabling this service the script executes
and maps everything as it should. 
 
One could certainly merely disable the IIS 6 Metabase Compatibility
after the scripts execution since everything has been configured and
internally mapped to IIS 7 configuration.
 
However, an IIS 7.0 compatible configuration script without the need to
enable optional modules and reboot the server would seem ideal.
 
My apologies if this has been brought up/addressed previously.
 
Dan Stusynski 

Re: isapi_redirector.vbs Tomcat Connector IIS 7 Compatibility

[Mladen Turk]

Stusynski, Dan wrote:
 
However, an IIS 7.0 compatible configuration script without the need to

enable optional modules and reboot the server would seem ideal.



I'll attend the Microsoft server 2008 compatibility lab
organized by Microsoft for ASF at the end of the February.
Guys that wrote that thing will be at our disposal
for Q&A, and one of the things will definitely be the
better IIS integration.


Regards,
Mladen


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

DO NOT REPLY [Bug 44238] - Tomcat NIO SSL connector can't handle large file

[bugzilla]

DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=44238





--- Additional Comments From [EMAIL PROTECTED]  2008-01-22 23:00 ---
This problem seems to have been fixed in the current trunk.

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

DO NOT REPLY [Bug 44281] New: - apache doesn't start when worker name is "*" in exclusion rule.

[bugzilla]

DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=44281

   Summary: apache doesn't start when worker name is "*" in
exclusion rule.
   Product: Tomcat 5
   Version: Nightly Build
  Platform: Other
OS/Version: Linux
Status: NEW
  Severity: normal
  Priority: P2
 Component: Native:JK
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]


hi

I tested mod_jk 1.2.27-dev (checkout from trunk head) on RHEL 4u5, i got 
segfault. 

When the worker name specified for the exclusion rule is "*", the apache doesn'
t start. 

ex.)
JkUnMount /*.jpg *
or
!/*.jpg=*

Index: mod_jk/native/common/jk_uri_worker_map.c
===
--- mod_jk/native/common/jk_uri_worker_map.c(revision 614422)
+++ mod_jk/native/common/jk_uri_worker_map.c(working copy)
@@ -468,6 +468,8 @@

 for (i = 0; i < IND_NEXT(uw_map->size); i++) {
 uri_worker_record_t *uwr = IND_NEXT(uw_map->maps)[i];
+if(uwr->match_type & MATCH_TYPE_NO_MATCH)
+continue;
 jk_worker_t *jw = wc_get_worker_for_name(uwr->worker_name, l);
 if (JK_IS_DEBUG_LEVEL(l))
 jk_log(l, JK_LOG_DEBUG,

My best regards. 

P.S.
The following patch was made because it was not displayed with status worker 
when the worker name was "*". 

Index: mod_jk/native/common/jk_status.c
===
--- mod_jk/native/common/jk_status.c(revision 614422)
+++ mod_jk/native/common/jk_status.c(working copy)
@@ -1358,7 +1358,7 @@
 if (uw_map) {
 for (i = 0; i < uw_map->size[uw_map->index]; i++) {
 uri_worker_record_t *uwr = uw_map->maps[uw_map->index][i];
-if (strcmp(uwr->worker_name, worker)) {
+if (strcmp(uwr->worker_name, worker)&&strcmp(uwr->worker_name, 
"*")) {
 continue;
 }
 count++;
@@ -1409,7 +1409,7 @@
 for (i = 0; i < uw_map->size[uw_map->index]; i++) {
 uri_worker_record_t *uwr = uw_map->maps[uw_map->index][i];

-if (strcmp(uwr->worker_name, worker)) {
+if (strcmp(uwr->worker_name, worker)&&strcmp(uwr->worker_name, "*")) {
 continue;
 }
 (*count_ptr)++;

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

DO NOT REPLY [Bug 44282] New: - WebappClassLoader.findClass calls getClassLoader without privileges

[bugzilla]

DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=44282

   Summary: WebappClassLoader.findClass calls getClassLoader without
privileges
   Product: Tomcat 5
   Version: 5.5.25
  Platform: Other
OS/Version: Linux
Status: NEW
  Severity: minor
  Priority: P4
 Component: Catalina
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]


When logging is in TRACE mode or lower,
org.apache.catalina.loader.WebappClassLoader.findClass(String) calls
getClassLoader() without a privileged block.  With security enabled, this will
cause a SecurityException if the RuntimePermission to getClassLoader is not 
granted.

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]