svn commit: r1439334 - /tomcat/trunk/webapps/docs/windows-auth-howto.xml

2013-01-28 Thread markt
Author: markt
Date: Mon Jan 28 10:34:24 2013
New Revision: 1439334

URL: http://svn.apache.org/viewvc?rev=1439334&view=rev
Log:
Add Jespa

Modified:
tomcat/trunk/webapps/docs/windows-auth-howto.xml

Modified: tomcat/trunk/webapps/docs/windows-auth-howto.xml
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/webapps/docs/windows-auth-howto.xml?rev=1439334&r1=1439333&r2=1439334&view=diff
==
--- tomcat/trunk/webapps/docs/windows-auth-howto.xml (original)
+++ tomcat/trunk/webapps/docs/windows-auth-howto.xml Mon Jan 28 10:34:24 2013
@@ -51,8 +51,9 @@ sections.
 
 
 
-This is a work in progress. There are a number of outstanding
-questions that require further testing. These include:
+This documentation is a work in progress. There are a number of
+outstanding questions around the edge cases that require further
+testing. These include:
 
 
 Does the domain name have to be in upper case?
@@ -219,7 +220,8 @@ com.sun.security.jgss.krb5.accept {
 
   
   Full details of this solution can be found through the
-  http://waffle.codeplex.com/";>Waffle site. The key features 
are:
+  http://waffle.codeplex.com/";>Waffle web site. The key features
+  are:
   
   Drop-in solution
   Simple configuration (no JAAS or Kerberos keytab configuration required)
@@ -231,7 +233,7 @@ com.sun.security.jgss.krb5.accept {
   
   Full details of this solution can be found through the
   http://static.springsource.org/spring-security/site/extensions/krb/index.html";>
-  Kerberos extension site. The key features are:
+  Kerberos extension web site. The key features are:
   
   Extension to Spring Security
   Requires a Kerberos keytab file to be generated
@@ -248,6 +250,16 @@ com.sun.security.jgss.krb5.accept {
   Pure Java solution
   
   
+
+  
+  Full details of this solution can be found through the
+  http://www.ioplex.com/";>project web siteThe key
+  features are:
+  
+  Pure Java solution
+  Advanced Active Directory integration
+  
+  
 
 
 



-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



svn commit: r1439335 - /tomcat/native/trunk/native/os/unix/system.c

2013-01-28 Thread rjung
Author: rjung
Date: Mon Jan 28 10:34:27 2013
New Revision: 1439335

URL: http://svn.apache.org/viewvc?rev=1439335&view=rev
Log:
Add CPU information to OS info for Linux.
This was already available under Windows and Solaris.

Forward port of r1350339 from 1.1.x branch.

Modified:
tomcat/native/trunk/native/os/unix/system.c

Modified: tomcat/native/trunk/native/os/unix/system.c
URL: 
http://svn.apache.org/viewvc/tomcat/native/trunk/native/os/unix/system.c?rev=1439335&r1=1439334&r2=1439335&view=diff
==
--- tomcat/native/trunk/native/os/unix/system.c (original)
+++ tomcat/native/trunk/native/os/unix/system.c Mon Jan 28 10:34:27 2013
@@ -127,6 +127,15 @@ TCN_IMPLEMENT_CALL(jint, OS, info)(TCN_S
 if (sysinfo(&info))
 rv = apr_get_os_error();
 else {
+static char buf[1024];
+unsigned long user = 0;
+unsigned long system = 0;
+long idle = 0;
+long long starttime = 0;
+int fd;
+int len;
+long sys_clk_tck = sysconf(_SC_CLK_TCK); /* number of system ticks 
per second */
+
 pvals[0] = (jlong)(info.totalram  * info.mem_unit);
 pvals[1] = (jlong)(info.freeram   * info.mem_unit);
 pvals[2] = (jlong)(info.totalswap * info.mem_unit);
@@ -134,6 +143,54 @@ TCN_IMPLEMENT_CALL(jint, OS, info)(TCN_S
 pvals[4] = (jlong)(info.sharedram * info.mem_unit);
 pvals[5] = (jlong)(info.bufferram * info.mem_unit);
 pvals[6] = (jlong)(100 - (info.freeram * 100 / info.totalram));
+
+if (sys_clk_tck >= 0) {
+/* Get total CPU times from /proc/stat */
+/* Example for the first line: cpu  2095497 8176 3280198 
908667841 1543576 28867 375399 0 0 */
+/* Accoring to the man pages, the numbers are given in units 
of USER_HZ:
+ * user mode, user mode with low priority (nice), system mode, 
and the idle task.
+ * Additional values can be ignored. */
+fd = open("/proc/stat", O_RDONLY);
+if (fd != -1) {
+len = read(fd, buf, sizeof buf - 1);
+if (len > 0) {
+buf[len] = '\0';
+if (sscanf(buf, "cpu %lu %*d %lu %ld", &user, &system, 
&idle) == 3) {
+pvals[7] = (jlong)(idle * 1000 / sys_clk_tck * 
1000); /* Idle Time in microseconds */
+pvals[8] = (jlong)(system * 1000 / sys_clk_tck * 
1000); /* Kernel Time in microseconds */
+pvals[9] = (jlong)(user * 1000 / sys_clk_tck * 
1000); /* User Time in microseconds */
+}
+}
+close(fd);
+}
+/* Get process CPU times from /proc/self/stat */
+/* Example for the first line:
+ * 6309 (csh) S 6308 6309 6309 34816 7124 4202496 15119 252261 
1 30 21 58 1537 1447 20 0 1 0 916031966 ... */
+/* Parsing it according to man -s 5 proci:
+ * pid %d, comm %s, state %c, ppid %d pgrp %d, session %d, 
tty_nr %d, tpgid %d, flags %u,
+ * minflt %lu, cminflt %lu,  majflt %lu, cmajflt %lu,
+ * utime %lu (!), stime %lu (!), cutime %ld (!), cstime %ld 
(!),
+ * priority %ld, nice %ld, num_threads %ld, itrealvalue %ld,
+ * starttime %llu (!) */
+fd = open("/proc/self/stat", O_RDONLY);
+if (fd != -1) {
+len = read(fd, buf, sizeof buf - 1);
+if (len > 0) {
+buf[len] = '\0';
+if (sscanf(buf, "%*d %*s %*c %*d %*d %*d %*d %*d %*u"
+" %*u %*u %*u %*u"
+" %lu %lu %*d %*d"
+" %*d %*d %*d %*d"
+"%llu", &user, &system, &starttime) == 
3) {
+pvals[10] = (jlong)(apr_time_now() - 
apr_time_make(info.uptime - starttime / sys_clk_tck, 0)); /* Process creation 
time (apr_time_t) */
+pvals[11] = (jlong)(system * 1000 / sys_clk_tck * 
1000); /* Process System Time in microseconds */
+pvals[12] = (jlong)(user * 1000 / sys_clk_tck * 
1000); /* Process User Time in microseconds */
+}
+}
+close(fd);
+}
+}
+
 rv = APR_SUCCESS;
 }
 }



-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



svn commit: r1439337 - /tomcat/native/trunk/native/os/unix/system.c

2013-01-28 Thread rjung
Author: rjung
Date: Mon Jan 28 10:55:45 2013
New Revision: 1439337

URL: http://svn.apache.org/viewvc?rev=1439337&view=rev
Log:
Docs in OS.java tell us we want microseconds.

Linux impl already provides microsecs, so lets
switch Solaris impl also to micro instead of milli.

Modified:
tomcat/native/trunk/native/os/unix/system.c

Modified: tomcat/native/trunk/native/os/unix/system.c
URL: 
http://svn.apache.org/viewvc/tomcat/native/trunk/native/os/unix/system.c?rev=1439337&r1=1439336&r2=1439337&view=diff
==
--- tomcat/native/trunk/native/os/unix/system.c (original)
+++ tomcat/native/trunk/native/os/unix/system.c Mon Jan 28 10:55:45 2013
@@ -210,8 +210,8 @@ TCN_IMPLEMENT_CALL(jint, OS, info)(TCN_S
 int res = 0;   /* general result state */
 /* non-static variables - sysinfo/swapctl use */
 long ret_sysconf;  /* value returned from sysconf 
call */
-long tck_dividend; /* factor used by transforming 
tick numbers to milliseconds */
-long tck_divisor;  /* divisor used by transforming 
tick numbers to milliseconds */
+long tck_dividend; /* factor used by transforming 
tick numbers to microseconds */
+long tck_divisor;  /* divisor used by transforming 
tick numbers to microseconds */
 long sys_pagesize = sysconf(_SC_PAGESIZE); /* size of a system memory 
page in bytes */
 long sys_clk_tck = sysconf(_SC_CLK_TCK); /* number of system ticks per 
second */
 struct anoninfo info;  /* structure for information 
about sizes in anonymous memory system */
@@ -283,11 +283,11 @@ TCN_IMPLEMENT_CALL(jint, OS, info)(TCN_S
 creation = (long)(now - (prusg.pr_tstamp.tv_sec -
  prusg.pr_create.tv_sec));
 }
-pvals[10] = (jlong)(creation);
-pvals[11] = (jlong)((jlong)prusg.pr_stime.tv_sec * 1000 +
-(prusg.pr_stime.tv_nsec / 100));
-pvals[12] = (jlong)((jlong)prusg.pr_utime.tv_sec * 1000 +
-(prusg.pr_utime.tv_nsec / 100));
+pvals[10] = (jlong)(creation * 100L);
+pvals[11] = (jlong)((jlong)prusg.pr_stime.tv_sec * 100L +
+(prusg.pr_stime.tv_nsec / 1000L));
+pvals[12] = (jlong)((jlong)prusg.pr_utime.tv_sec * 100L +
+(prusg.pr_utime.tv_nsec / 1000L));
 pvals[15] = (jlong)(prusg.pr_majf);
 }
 else {
@@ -299,7 +299,7 @@ TCN_IMPLEMENT_CALL(jint, OS, info)(TCN_S
 rv = apr_get_os_error();
 }
 else {
-tck_dividend = 1000;
+tck_dividend = 100L;
 tck_divisor = sys_clk_tck;
 for (i = 0; i < 3; i++) {
 if (tck_divisor % 2 == 0) {



-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



svn commit: r1439338 - in /tomcat/site/trunk: docs/security-6.html docs/security-7.html xdocs/security-6.xml xdocs/security-7.xml

2013-01-28 Thread markt
Author: markt
Date: Mon Jan 28 10:57:09 2013
New Revision: 1439338

URL: http://svn.apache.org/viewvc?rev=1439338&view=rev
Log:
Add CVE-2012-5568 to the not a vulnerability in Tomcat section.

Modified:
tomcat/site/trunk/docs/security-6.html
tomcat/site/trunk/docs/security-7.html
tomcat/site/trunk/xdocs/security-6.xml
tomcat/site/trunk/xdocs/security-7.xml

Modified: tomcat/site/trunk/docs/security-6.html
URL: 
http://svn.apache.org/viewvc/tomcat/site/trunk/docs/security-6.html?rev=1439338&r1=1439337&r2=1439338&view=diff
==
--- tomcat/site/trunk/docs/security-6.html (original)
+++ tomcat/site/trunk/docs/security-6.html Mon Jan 28 10:57:09 2013
@@ -1791,6 +1791,33 @@
 
 
 
+Low: Denial Of Service
+   http://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2012-5568"; 
rel="nofollow">CVE-2012-5568
+
+
+
+Sending an HTTP request 1 byte at a time will consume a thread from the
+   connection pool until the request has been fully processed if using the
+   BIO or APR/native HTTP connectors. Multiple requests may be used to
+   consume all threads in the connection pool thereby creating a denial of
+   service.
+
+
+Since the relationship between the client side resources and server side
+   resources is a linear one, this issue is not something that the Tomcat
+   Security Team views as a vulnerability. This is a generic DoS problem 
and
+   there is no magic solution. This issue has been discussed several times
+   on the Tomcat mailing lists. The best place to start to review these
+   discussions is the report for
+   https://issues.apache.org/bugzilla/show_bug.cgi?id=54263";>bug
+   54236.
+
+
+This was first discussed on the public Tomcat users mailing list on 19
+   June 2009.
+
+
+
 Important: Remote Denial Of Service
http://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2010-4476"; 
rel="nofollow">CVE-2010-4476
 

Modified: tomcat/site/trunk/docs/security-7.html
URL: 
http://svn.apache.org/viewvc/tomcat/site/trunk/docs/security-7.html?rev=1439338&r1=1439337&r2=1439338&view=diff
==
--- tomcat/site/trunk/docs/security-7.html (original)
+++ tomcat/site/trunk/docs/security-7.html Mon Jan 28 10:57:09 2013
@@ -1395,6 +1395,36 @@
   
 
 
+Low: Denial Of Service
+   http://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2012-5568"; 
rel="nofollow">CVE-2012-5568
+
+
+
+Sending an HTTP request 1 byte at a time will consume a thread from the
+   connection pool until the request has been fully processed if using the
+   BIO or APR/native HTTP connectors. Multiple requests may be used to
+   consume all threads in the connection pool thereby creating a denial of
+   service.
+
+
+Since the relationship between the client side resources and server side
+   resources is a linear one, this issue is not something that the Tomcat
+   Security Team views as a vulnerability. This is a generic DoS problem 
and
+   there is no magic solution. This issue has been discussed several times
+   on the Tomcat mailing lists. The best place to start to review these
+   discussions is the report for
+   https://issues.apache.org/bugzilla/show_bug.cgi?id=54263";>bug
+   54236.
+
+
+This was first discussed on the public Tomcat users mailing list on 19
+   June 2009.
+
+
+Affects: 7.0.0-7.0.x
+
+
+
 Important: Remote Denial Of Service
http://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2010-4476"; 
rel="nofollow">CVE-2010-4476
 

Modified: tomcat/site/trunk/xdocs/security-6.xml
URL: 
http://svn.apache.org/viewvc/tomcat/site/trunk/xdocs/security-6.xml?rev=1439338&r1=1439337&r2=1439338&view=diff
==
--- tomcat/site/trunk/xdocs/security-6.xml (original)
+++ tomcat/site/trunk/xdocs/security-6.xml Mon Jan 28 10:57:09 2013
@@ -909,6 +909,27 @@
 
   
 
+Low: Denial Of Service
+   CVE-2012-5568
+
+Sending an HTTP request 1 byte at a time will consume a thread from the
+   connection pool until the request has been fully processed if using the
+   BIO or APR/native HTTP connectors. Multiple requests may be used to
+   consume all threads in the connection pool thereby creating a denial of
+   service.
+
+Since the relationship between the client side resources and server side
+   resources is a linear one, this issue is not something that the Tomcat
+   Security Team views as a vulnerability. This is a generic DoS problem 
and
+   there is no magic solution. This issue has been discussed several times
+   on the Tomcat mailing lists. The best place to start to review these
+   discussions is the report for
+   https://issues.apache.org/bugzilla/show_bug.cgi?id=54263";>bug
+   54236.
+
+This was fir

svn commit: r1439341 - in /tomcat/native/branches/1.1.x: ./ native/os/unix/system.c xdocs/miscellaneous/changelog.xml

2013-01-28 Thread rjung
Author: rjung
Date: Mon Jan 28 11:00:09 2013
New Revision: 1439341

URL: http://svn.apache.org/viewvc?rev=1439341&view=rev
Log:
Docs in OS.java tell us we want microseconds.

Linux impl already provides microsecs, so lets
switch Solaris impl also to micro instead of milli.

Backport of 1439337 from trunk.

Modified:
tomcat/native/branches/1.1.x/   (props changed)
tomcat/native/branches/1.1.x/native/os/unix/system.c
tomcat/native/branches/1.1.x/xdocs/miscellaneous/changelog.xml

Propchange: tomcat/native/branches/1.1.x/
--
  Merged /tomcat/native/trunk:r1439337

Modified: tomcat/native/branches/1.1.x/native/os/unix/system.c
URL: 
http://svn.apache.org/viewvc/tomcat/native/branches/1.1.x/native/os/unix/system.c?rev=1439341&r1=1439340&r2=1439341&view=diff
==
--- tomcat/native/branches/1.1.x/native/os/unix/system.c (original)
+++ tomcat/native/branches/1.1.x/native/os/unix/system.c Mon Jan 28 11:00:09 
2013
@@ -210,8 +210,8 @@ TCN_IMPLEMENT_CALL(jint, OS, info)(TCN_S
 int res = 0;   /* general result state */
 /* non-static variables - sysinfo/swapctl use */
 long ret_sysconf;  /* value returned from sysconf 
call */
-long tck_dividend; /* factor used by transforming 
tick numbers to milliseconds */
-long tck_divisor;  /* divisor used by transforming 
tick numbers to milliseconds */
+long tck_dividend; /* factor used by transforming 
tick numbers to microseconds */
+long tck_divisor;  /* divisor used by transforming 
tick numbers to microseconds */
 long sys_pagesize = sysconf(_SC_PAGESIZE); /* size of a system memory 
page in bytes */
 long sys_clk_tck = sysconf(_SC_CLK_TCK); /* number of system ticks per 
second */
 struct anoninfo info;  /* structure for information 
about sizes in anonymous memory system */
@@ -283,11 +283,11 @@ TCN_IMPLEMENT_CALL(jint, OS, info)(TCN_S
 creation = (long)(now - (prusg.pr_tstamp.tv_sec -
  prusg.pr_create.tv_sec));
 }
-pvals[10] = (jlong)(creation);
-pvals[11] = (jlong)((jlong)prusg.pr_stime.tv_sec * 1000 +
-(prusg.pr_stime.tv_nsec / 100));
-pvals[12] = (jlong)((jlong)prusg.pr_utime.tv_sec * 1000 +
-(prusg.pr_utime.tv_nsec / 100));
+pvals[10] = (jlong)(creation * 100L);
+pvals[11] = (jlong)((jlong)prusg.pr_stime.tv_sec * 100L +
+(prusg.pr_stime.tv_nsec / 1000L));
+pvals[12] = (jlong)((jlong)prusg.pr_utime.tv_sec * 100L +
+(prusg.pr_utime.tv_nsec / 1000L));
 pvals[15] = (jlong)(prusg.pr_majf);
 }
 else {
@@ -299,7 +299,7 @@ TCN_IMPLEMENT_CALL(jint, OS, info)(TCN_S
 rv = apr_get_os_error();
 }
 else {
-tck_dividend = 1000;
+tck_dividend = 100L;
 tck_divisor = sys_clk_tck;
 for (i = 0; i < 3; i++) {
 if (tck_divisor % 2 == 0) {

Modified: tomcat/native/branches/1.1.x/xdocs/miscellaneous/changelog.xml
URL: 
http://svn.apache.org/viewvc/tomcat/native/branches/1.1.x/xdocs/miscellaneous/changelog.xml?rev=1439341&r1=1439340&r2=1439341&view=diff
==
--- tomcat/native/branches/1.1.x/xdocs/miscellaneous/changelog.xml (original)
+++ tomcat/native/branches/1.1.x/xdocs/miscellaneous/changelog.xml Mon Jan 28 
11:00:09 2013
@@ -36,6 +36,14 @@
   new documentation project for Tomcat Native was started.
   
 
+
+  
+
+  Switch CPU information on Solaris from milliseconds to
+  microseconds. Make consistent with OS.java and Linux impl. (rjung)
+
+  
+
 
   
 



-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



[Bug 52318] Version in tomcat-jdbc POM is conflicted with Version in MANIFEST for JULI JAR

2013-01-28 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=52318

--- Comment #17 from Martin Lichtin  ---
Created attachment 29897
  --> https://issues.apache.org/bugzilla/attachment.cgi?id=29897&action=edit
Updated dependency on Juli, allow versions 7.0.x

Could this be fixed?
I would like to use tomcat-jdbc in an OSGi environment and currently the
version restriction [6.0.18, 7.1.0) on juli.logging forces me to install
a version 6 of tomcat-juli.

-- 
You are receiving this mail because:
You are the assignee for the bug.

-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



[Bug 52318] Version in tomcat-jdbc POM is conflicted with Version in MANIFEST for JULI JAR

2013-01-28 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=52318

Martin Lichtin  changed:

   What|Removed |Added

 CC||lich...@yahoo.com

-- 
You are receiving this mail because:
You are the assignee for the bug.

-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



buildbot success in ASF Buildbot on tomcat-trunk

2013-01-28 Thread buildbot
The Buildbot has detected a restored build on builder tomcat-trunk while 
building ASF Buildbot.
Full details are available at:
 http://ci.apache.org/builders/tomcat-trunk/builds/3845

Buildbot URL: http://ci.apache.org/

Buildslave for this Build: bb-vm_ubuntu

Build Reason: scheduler
Build Source Stamp: [branch tomcat/trunk] 1439334
Blamelist: markt

Build succeeded!

sincerely,
 -The Buildbot





svn commit: r1439345 - /tomcat/native/branches/1.1.x/native/src/sslutils.c

2013-01-28 Thread rjung
Author: rjung
Date: Mon Jan 28 11:19:50 2013
New Revision: 1439345

URL: http://svn.apache.org/viewvc?rev=1439345&view=rev
Log:
Fix unused variable warning.

Modified:
tomcat/native/branches/1.1.x/native/src/sslutils.c

Modified: tomcat/native/branches/1.1.x/native/src/sslutils.c
URL: 
http://svn.apache.org/viewvc/tomcat/native/branches/1.1.x/native/src/sslutils.c?rev=1439345&r1=1439344&r2=1439345&view=diff
==
--- tomcat/native/branches/1.1.x/native/src/sslutils.c (original)
+++ tomcat/native/branches/1.1.x/native/src/sslutils.c Mon Jan 28 11:19:50 2013
@@ -902,7 +902,6 @@ static int add_ocsp_cert(OCSP_REQUEST **
 */
 static apr_socket_t *make_socket(char *hostname, int port, apr_pool_t *mp)
 {
-int r = 0;
 apr_sockaddr_t *sa_in;
 apr_status_t status;
 apr_socket_t *sock = NULL;



-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



Confusion around tcnative

2013-01-28 Thread Rainer Jung
I'm a bit confused about tcnative and the various trunks and branches:

1) tcnative trunk vs. branch 1.1.x.

It seems most updates are directly done to the branch. So trunk is
outdated. On the other hand originally there were some updates to trunk
not in 1.1.x. I think this should be cleaned up. The current trunk
doesn't seem to be useful.

2) tcnative and Java classes

We still have Java classes in the tcnative branch. They neither match
with the copies maintained in the tc7 branch, nor with the ones in the
tc trunk branch. tcnative trunk does not have the Java files.

Is there any use for the copies of the Java files in the tcnative 1.1.x
branch? Otherwise I think we should remov them and check them out from
TC7 or TC trunk when rolling a tcnative release.

Note that tcnative also contains test and examples Java classes, which
are *not* also in the TC branches. Those must be kept in tcnative (or
moved to TC).

Regards,

Rainer

-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



svn commit: r1439347 - /tomcat/trunk/test/org/apache/tomcat/websocket/TestWsWebSocketContainer.java

2013-01-28 Thread markt
Author: markt
Date: Mon Jan 28 11:20:48 2013
New Revision: 1439347

URL: http://svn.apache.org/viewvc?rev=1439347&view=rev
Log:
Drop 10 10s. 100s was a local change while I was debugging so I had more time.

Modified:
tomcat/trunk/test/org/apache/tomcat/websocket/TestWsWebSocketContainer.java

Modified: 
tomcat/trunk/test/org/apache/tomcat/websocket/TestWsWebSocketContainer.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/test/org/apache/tomcat/websocket/TestWsWebSocketContainer.java?rev=1439347&r1=1439346&r2=1439347&view=diff
==
--- tomcat/trunk/test/org/apache/tomcat/websocket/TestWsWebSocketContainer.java 
(original)
+++ tomcat/trunk/test/org/apache/tomcat/websocket/TestWsWebSocketContainer.java 
Mon Jan 28 11:20:48 2013
@@ -79,7 +79,7 @@ public class TestWsWebSocketContainer ex
 wsSession.addMessageHandler(handler);
 wsSession.getRemote().sendString(MESSAGE_STRING_1);
 
-boolean latchResult = handler.getLatch().await(100, TimeUnit.SECONDS);
+boolean latchResult = handler.getLatch().await(10, TimeUnit.SECONDS);
 
 Assert.assertTrue(latchResult);
 
@@ -223,7 +223,7 @@ public class TestWsWebSocketContainer ex
 
wsSession.getRemote().sendBytes(ByteBuffer.wrap(MESSAGE_BINARY_4K));
 }
 
-boolean latchResult = handler.getLatch().await(100, TimeUnit.SECONDS);
+boolean latchResult = handler.getLatch().await(10, TimeUnit.SECONDS);
 
 Assert.assertTrue(latchResult);
 



-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



Re: svn commit: r1438747 - /tomcat/trunk/test/org/apache/tomcat/websocket/TestWsWebSocketContainer.java

2013-01-28 Thread Mark Thomas
On 26/01/2013 12:30, Konstantin Kolinko wrote:
> 2013/1/26 Mark Thomas :
>> kkoli...@apache.org wrote:
>>
>>> Author: kkolinko
>>> Date: Fri Jan 25 22:34:57 2013
>>> New Revision: 1438747
>>>
>>> URL: http://svn.apache.org/viewvc?rev=1438747&view=rev
>>> Log:
>>> Make the messages list synchronized as a whole, instead of just using a
>>> volatile reference to it.
>>> I am still observing random failures with TestWsWebSocketContainer, so
>>> an issue is not here.
>>
>> So why make the change?
>>
> 
> Because of
> http://svn.apache.org/viewvc?view=revision&revision=1437930

OK. The log message seemed to suggest the change was pointless. Hence my
question.

> If you just need to propagate the "messages" reference across threads,
> then I think r1437930 should have been more simple: just marking the
> field as "final".
> If you are protecting access to inner structures of ArrayList class, I
> think that marking the reference as volatile is not enough and that
> using a synchronized list is more adequate.
> 
> I think "volatile" modifier applies to accessing the filed itself, and
> has no effect when you use the value returned by getMessages() or
> store it in a local variable.
> 
> Well, I do not see where a concurrency can come from here, so mere
> "final" should be enough. The latch did count down (otherwise
> assertTrue(latchResult) detected a failure) so either a message was
> received or TesterEndpoint.onClose() or TesterEndpoint.onError() was
> called. The latch should provide synchronization between threads.

Having just read up on "happens-before" I do not believe the list needs
to be synchronised. I believe the root cause of the problem I was trying
to solve was fixed in r1437930 when I switched the order of adding the
message to the list and counting down the latch.

Mark


-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



svn commit: r1439351 - /tomcat/trunk/test/org/apache/tomcat/websocket/TestWsWebSocketContainer.java

2013-01-28 Thread markt
Author: markt
Date: Mon Jan 28 11:32:26 2013
New Revision: 1439351

URL: http://svn.apache.org/viewvc?rev=1439351&view=rev
Log:
Same ends, different means.

Modified:
tomcat/trunk/test/org/apache/tomcat/websocket/TestWsWebSocketContainer.java

Modified: 
tomcat/trunk/test/org/apache/tomcat/websocket/TestWsWebSocketContainer.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/test/org/apache/tomcat/websocket/TestWsWebSocketContainer.java?rev=1439351&r1=1439350&r2=1439351&view=diff
==
--- tomcat/trunk/test/org/apache/tomcat/websocket/TestWsWebSocketContainer.java 
(original)
+++ tomcat/trunk/test/org/apache/tomcat/websocket/TestWsWebSocketContainer.java 
Mon Jan 28 11:32:26 2013
@@ -19,9 +19,8 @@ package org.apache.tomcat.websocket;
 import java.io.IOException;
 import java.net.URI;
 import java.nio.ByteBuffer;
-import java.util.ArrayList;
-import java.util.Collections;
 import java.util.List;
+import java.util.concurrent.CopyOnWriteArrayList;
 import java.util.concurrent.CountDownLatch;
 import java.util.concurrent.TimeUnit;
 
@@ -246,8 +245,7 @@ public class TestWsWebSocketContainer ex
 
 private final CountDownLatch latch;
 
-private final List messages = Collections
-.synchronizedList(new ArrayList());
+private final List messages = new CopyOnWriteArrayList<>();
 
 public TesterMessageHandler(CountDownLatch latch) {
 this.latch = latch;



-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



Re: [VOTE] Release Apache Tomcat Native 1.1.26

2013-01-28 Thread Rainer Jung
On 25.01.2013 17:01, jean-frederic clere wrote:
> Version 1.1.26 is bug fixing release.
> The proposed release artefacts can be found at [1],
> and the build was done using tag [2].
> 
> The VOTE will remain open for at least 48 hours.
> 
> The Apache Tomcat Native 1.1.26 is
>  [X] Stable, go ahead and release
>  [ ] Broken because of ...

+1 to release

Detailed results ("-" indicates things which we could improve). Though
the list is long, it is again shorter than when I checked last time
(1.1.24). Overall I'm still +1 for stable, because I didn't find
regressions. But there is room for improvement.

+ Tested with Java 1.6.0_37 (but configured against Java 1.5.0_22),
  APR 1.4.6 and OpenSSL 1.0.1c
+ Platforms Solaris 8+10 Sparc, SLES 10, 32 and 64 Bits,
  SLES 11 64 Bits, RHEL 5+6 64 Bits
+ MD5 OK
+ signatures OK
+ key in KEYS file
- File 'bin/tcnative-1-src.pdb' missing in the binaries zips
- name of ocsp binary changed from
  ...ocsp-win32... to ...win32-ocsp...
  (changed back to how it was before 1.1.24).
+ gz and zip for sources consistent
- Except for different permissions: zip seems to also contain group
  write permissions. Not a real problem, but it's a bit strange
  that perms differ between the archive (no regression).
+ source dist consistent with svn tag
- config-guess and config.sub should get an update this year
  ('2011-05-11' resp. '2011-03-23' instead of recent '2012-12-30'
   resp. '2013-01-23'). They are copied from the APR source tree
  when running buildconf.
+ recreated release with jnirelease script, results are
  consistent with source dist, except for minor expected diffs in
  CHANGELOG.txt, build-outputs.mk and generated docs
  (whitespace and attribute ordering)
+ make succeeds and builds lib (no warnings)
- most unit tests contained in TC 7 headrun successful with
  APR connector and this version of tcnative
  Except:
  - TestMaxConnections: failed on several platforms, but also for
non-apr connectors. Not reproducible
  - TestStandardWrapper: Crash on Solaris 8 during shutdown
Stack see end of mail.
- ant part of build (no regressions, cited from 1.1.24 tests):
  - No mentioning of running "ant download" before tests. Without
it test compilation fails.
  - "ant test" fails in line 85 of SocketServerTestSuite.java, because
on my system the checking for precisely 2 milliseconds won't work.
The call returns after 11 millis not after 2
  - "ant run-echo": will fail, because by default uses privileged
port 23. Maybe switch to 8023? Users should not run tests as root.
  - "ant run-ssl-server": Could't we include a test certificates in the
test folder? What should the test produce, if run successfully?
  - "ant run-local-server": Creates a unix socket "\\.\PIPE\test" in the
examples directory, then waits. How is the test expected to work?
And the file name doesn't seem to be appropriate for Unix.
  - run-echo, run-ssl-server and run-local-server: I couldn't figure
out, what those were actually supposed to show (what is a positive
result vs. a negative one).

Last time sebb responded to the announcement mail:

- 8>< ---
Please include a brief synopsis (1 or 2 sentences) of the purpose of
the TLP/Product in all announcements sent outside the TLP mailing
lists.

The developers and users will (presumably) know what the product is
about, but others are unlikely to know.

S.
P.S. Just about every other TLP includes this information, including HTTD...
- 8>< ---

You might want to add such type of info to the announcement for 1.1.26.

Stack of crash during unit test on Solaris 8 (non-reproducible):

C  [libapr-1.so.0+0x14d54]  apr_allocator_destroy+0x10
C  [libapr-1.so.0+0x15a7c]  apr_pool_terminate+0x4c
j  org.apache.tomcat.jni.Library.terminate()V+0
j  org.apache.tomcat.jni.Library.terminate()V+0
v  ~StubRoutines::call_stub
V  [libjvm.so+0xe8ff8]  void
JavaCalls::call_helper(JavaValue*,methodHandle*,JavaCallArguments*,Thread*)+0x218
V  [libjvm.so+0x506c94]
oopDesc*Reflection::invoke(instanceKlassHandle,methodHandle,Handle,bool,objArrayHandle,BasicType,objArrayHandle,bool,Thread*)+0x13a8
V  [libjvm.so+0x178048]
oopDesc*Reflection::invoke_method(oopDesc*,Handle,objArrayHandle,Thread*)+0x260
V  [libjvm.so+0x177a9c]  JVM_InvokeMethod+0x350
C  [libjava.so+0x10d6c]
Java_sun_reflect_NativeMethodAccessorImpl_invoke0+0x18
j
sun.reflect.NativeMethodAccessorImpl.invoke0(Ljava/lang/reflect/Method;Ljava/lang/Object;[Ljava/lang/Object;)Ljava/lang/Object;+0
j
sun.reflect.NativeMethodAccessorImpl.invoke0(Ljava/lang/reflect/Method;Ljava/lang/Object;[Ljava/lang/Object;)Ljava/lang/Object;+0
j
sun.reflect.NativeMethodAccessorImpl.invoke(Ljava/lang/Object;[Ljava/lang/Object;)Ljava/lang/Object;+87
J
sun.reflect.DelegatingMethodAccessorImpl.invoke(Ljava/lang/Object;[Ljava/lang/Object;)Ljava/lang/Object;
J
java.lang.reflect.Method.invoke(Ljava/lang/Object;[Ljava/lang/Object;)Ljava/lang/Object;
j  org.apache.catalina.core.AprLife

[Tomcat Wiki] Trivial Update of "LinnieNow" by LinnieNow

2013-01-28 Thread Apache Wiki
Dear Wiki user,

You have subscribed to a wiki page or wiki category on "Tomcat Wiki" for change 
notification.

The "LinnieNow" page has been changed by LinnieNow:
http://wiki.apache.org/tomcat/LinnieNow

New page:
I am 22 years old and my name is Bernardo Kowalski. I life in Gonten 
(Switzerland).<><><>My blog post 
[[http://www.oesterreich-online.info/index.php?go=valid&user=c82d264570c391f71cdb14eb647ea886|perfect
 keylogger]]

-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



Re: Time for Taglibs to be sent to the archive?

2013-01-28 Thread Mark Thomas
On 26/01/2013 21:51, Henri Yandell wrote:
> On Fri, Jan 18, 2013 at 8:30 AM, Jeremy Boynes  wrote:
>> On Jan 18, 2013, at 1:34 AM, Konstantin Kolinko wrote:
>>> Regarding the two taglibs that are not yet in the attic, I have no
>>> interest in "RDC" taglib, but I am interested in JSTL one.
>>
>> +1
>>
>>>
>>> I think once we make the first release, things should go easier after that.
>>>
>>> A few notes after quick review of the sources:
>>>
>>> 1. Can we go up from Java 5 and require/use at least Java 6 to build
>>> the project?
>>>
>>> I am even OK to be brave and go up to Java 7.
>>>
>>> I do not like Java 5, because
>>> a) It is outdated.
>>> It would be strange for a "new" project to use that if we are going
>>> to support it for long.
>>> b) It is not opensource.
>>> OpenJDK is since Java 6.
>>>
>>> The version of Java is important for this class, that implements
>>> javax.sql.DataSource:
>>>
>>> \standard\impl\src\main\java\org\apache\taglibs\standard\tag\common\sql\DataSourceWrapper.java
>>>
>>> Java 7 will allow us to support a later version of JDBC and will allow
>>> this project to build on Gump.
>>
>> There is also an issue with the I18N tags taking a long time to start on a 
>> Java6 platform due to changes in the way Locales are located by the JRE. I 
>> remember some discussion on fixing this but a requirement to stay on Java 5 
>> meant having to build two implementations and switch between them which I'd 
>> planned to do once a release was out. If we pre-req 6 or 7 then the 
>> implementation can just be updated and this issue closed easier.
> 
> Would the TCK pass if it was on Java 7?

We are talking about Taglibs 1.2 right? (If not adjust versions below
accordingly).

Taglibs 1.2 requires JSP 2.1.

JSP 2.1 requires Java 5 or later (as does Servlet 2.5 and J2EE 5).

Based on that, I would say that the TCK has to pass when running on Java 5.

/me goes looking for some TCK docs

I've just checked the latest version of the TCK documentation for JSTL
1.2 and while the TCK has been updated so it will run on Java 7, the
reference runtime remains Java 5. My interpretation of that is that the
TCK must pass when running with a Java 5 JRE.

Mark


-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



buildbot failure in ASF Buildbot on tomcat-trunk

2013-01-28 Thread buildbot
The Buildbot has detected a new failure on builder tomcat-trunk while building 
ASF Buildbot.
Full details are available at:
 http://ci.apache.org/builders/tomcat-trunk/builds/3846

Buildbot URL: http://ci.apache.org/

Buildslave for this Build: bb-vm_ubuntu

Build Reason: scheduler
Build Source Stamp: [branch tomcat/trunk] 1439347
Blamelist: markt

BUILD FAILED: failed compile_1

sincerely,
 -The Buildbot





Re: Confusion around tcnative

2013-01-28 Thread Mladen Turk

On 01/28/2013 12:20 PM, Rainer Jung wrote:

I'm a bit confused about tcnative and the various trunks and branches:

1) tcnative trunk vs. branch 1.1.x.

It seems most updates are directly done to the branch. So trunk is
outdated. On the other hand originally there were some updates to trunk
not in 1.1.x. I think this should be cleaned up. The current trunk
doesn't seem to be useful.



Agreed.
Trunk should be used for 1.2 and not backward compatible.
I plan to add different polling mechanism using interrupts instead
timeout polling (one of the reasons why split was made).
This is now possible with APR_WAKEABLE pollsets.
It would require rewriting Java part as well.

Initial plan was to finish that for Tomcat8, but not sure how
much time we still have before 8 gets released.



Regards
--
^TM

-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



[Tomcat Wiki] Update of "LocalBadContent" by KonstantinKolinko

2013-01-28 Thread Apache Wiki
Dear Wiki user,

You have subscribed to a wiki page or wiki category on "Tomcat Wiki" for change 
notification.

The "LocalBadContent" page has been changed by KonstantinKolinko:
http://wiki.apache.org/tomcat/LocalBadContent?action=diff&rev1=69&rev2=70

  nordenpark\.com
  novolinespielen\.org
  oa8000\.com\.cn
+ oesterreich-online\.info
  oro-compro\.com
  o-f\.com
  paisi\.com

-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



[Bug 54060] DigestAuthenticator doesn't parse Authorization header correctly

2013-01-28 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=54060

Konstantin Kolinko  changed:

   What|Removed |Added

 Status|RESOLVED|REOPENED
 Resolution|FIXED   |---

--- Comment #13 from Konstantin Kolinko  ---
REOPENing the issue to address Comment 12

-- 
You are receiving this mail because:
You are the assignee for the bug.

-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



Re: Confusion around tcnative

2013-01-28 Thread Rainer Jung
On 28.01.2013 13:33, Mladen Turk wrote:
> On 01/28/2013 12:20 PM, Rainer Jung wrote:
>> I'm a bit confused about tcnative and the various trunks and branches:
>>
>> 1) tcnative trunk vs. branch 1.1.x.
>>
>> It seems most updates are directly done to the branch. So trunk is
>> outdated. On the other hand originally there were some updates to trunk
>> not in 1.1.x. I think this should be cleaned up. The current trunk
>> doesn't seem to be useful.
>>
> 
> Agreed.
> Trunk should be used for 1.2 and not backward compatible.
> I plan to add different polling mechanism using interrupts instead
> timeout polling (one of the reasons why split was made).
> This is now possible with APR_WAKEABLE pollsets.
> It would require rewriting Java part as well.

But how do we clean up trunk? Throw away and start from new? I think
currently trunk misses things which are in the 1.1 branch, so building
something on top of the current trunk will be troublesome.

> Initial plan was to finish that for Tomcat8, but not sure how
> much time we still have before 8 gets released.

Regards,

Rainer


-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



Re: svn commit: r1438747 - /tomcat/trunk/test/org/apache/tomcat/websocket/TestWsWebSocketContainer.java

2013-01-28 Thread Konstantin Kolinko
2013/1/28 Mark Thomas :
> On 26/01/2013 12:30, Konstantin Kolinko wrote:
>> 2013/1/26 Mark Thomas :
>>> kkoli...@apache.org wrote:
>>>
 Author: kkolinko
 Date: Fri Jan 25 22:34:57 2013
 New Revision: 1438747

 URL: http://svn.apache.org/viewvc?rev=1438747&view=rev
 Log:
 Make the messages list synchronized as a whole, instead of just using a
 volatile reference to it.
 I am still observing random failures with TestWsWebSocketContainer, so
 an issue is not here.
>>>
>>> So why make the change?
>>>
>>
>> Because of
>> http://svn.apache.org/viewvc?view=revision&revision=1437930
>
> OK. The log message seemed to suggest the change was pointless. Hence my
> question.
>
>> If you just need to propagate the "messages" reference across threads,
>> then I think r1437930 should have been more simple: just marking the
>> field as "final".
>> If you are protecting access to inner structures of ArrayList class, I
>> think that marking the reference as volatile is not enough and that
>> using a synchronized list is more adequate.
>>
>> I think "volatile" modifier applies to accessing the filed itself, and
>> has no effect when you use the value returned by getMessages() or
>> store it in a local variable.
>>
>> Well, I do not see where a concurrency can come from here, so mere
>> "final" should be enough. The latch did count down (otherwise
>> assertTrue(latchResult) detected a failure) so either a message was
>> received or TesterEndpoint.onClose() or TesterEndpoint.onError() was
>> called. The latch should provide synchronization between threads.
>
> Having just read up on "happens-before" I do not believe the list needs
> to be synchronised.

Agreed.

I am OK with leaving it as is (as well as with CopyOnWriteArrayList
from r1439351). It should not matter for this test case.

> I believe the root cause of the problem I was trying
> to solve was fixed in r1437930 when I switched the order of adding the
> message to the list and counting down the latch.

Agreed. It should be it.
(For the record, you obviously meant r1438229 and the change in
onMessage(..) in TestWsWebSocketContainer.java)

BTW, the test still fails for BIO in the last two buildbot runs. (NIO was OK).

Best regards,
Konstantin Kolinko

-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



Re: svn commit: r1439334 - /tomcat/trunk/webapps/docs/windows-auth-howto.xml

2013-01-28 Thread Konstantin Kolinko
2013/1/28  :
> Author: markt
> Date: Mon Jan 28 10:34:24 2013
> New Revision: 1439334
>
> URL: http://svn.apache.org/viewvc?rev=1439334&view=rev
> Log:
> Add Jespa
>
> Modified:
> tomcat/trunk/webapps/docs/windows-auth-howto.xml
>
> Modified: tomcat/trunk/webapps/docs/windows-auth-howto.xml
> URL: 
> http://svn.apache.org/viewvc/tomcat/trunk/webapps/docs/windows-auth-howto.xml?rev=1439334&r1=1439333&r2=1439334&view=diff
> ==
> --- tomcat/trunk/webapps/docs/windows-auth-howto.xml (original)
> +++ tomcat/trunk/webapps/docs/windows-auth-howto.xml Mon Jan 28 10:34:24 2013
> @@ -51,8 +51,9 @@ sections.
>  
>
>  
> -This is a work in progress. There are a number of outstanding
> -questions that require further testing. These include:
> +This documentation is a work in progress. There are a number of
> +outstanding questions around the edge cases that require further
> +testing. These include:
>  
>  
>  Does the domain name have to be in upper case?
> @@ -219,7 +220,8 @@ com.sun.security.jgss.krb5.accept {
>
>
>Full details of this solution can be found through the
> -  http://waffle.codeplex.com/";>Waffle site. The key features 
> are:
> +  http://waffle.codeplex.com/";>Waffle web site. The key features


Maybe it would be better with a  rel="nofollow"  like elsewhere on
tomcat.apache.org.

This fix has not been merged to 7.0.x yet

> +  are:
>
>Drop-in solution
>Simple configuration (no JAAS or Kerberos keytab configuration 
> required)
> @@ -231,7 +233,7 @@ com.sun.security.jgss.krb5.accept {
>
>Full details of this solution can be found through the
> href="http://static.springsource.org/spring-security/site/extensions/krb/index.html";>
> -  Kerberos extension site. The key features are:
> +  Kerberos extension web site. The key features are:
>
>Extension to Spring Security
>Requires a Kerberos keytab file to be generated
> @@ -248,6 +250,16 @@ com.sun.security.jgss.krb5.accept {
>Pure Java solution
>
>
> +
> +  
> +  Full details of this solution can be found through the
> +  http://www.ioplex.com/";>project web siteThe key
> +  features are:
> +  
> +  Pure Java solution
> +  Advanced Active Directory integration
> +  
> +  
>  
>
>  
>
>
>
> -
> To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
> For additional commands, e-mail: dev-h...@tomcat.apache.org
>

-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



Re: Confusion around tcnative

2013-01-28 Thread Mladen Turk

On 01/28/2013 01:45 PM, Rainer Jung wrote:

On 28.01.2013 13:33, Mladen Turk wrote:

On 01/28/2013 12:20 PM, Rainer Jung wrote:

I'm a bit confused about tcnative and the various trunks and branches:

1) tcnative trunk vs. branch 1.1.x.

It seems most updates are directly done to the branch. So trunk is
outdated. On the other hand originally there were some updates to trunk
not in 1.1.x. I think this should be cleaned up. The current trunk
doesn't seem to be useful.



Agreed.
Trunk should be used for 1.2 and not backward compatible.
I plan to add different polling mechanism using interrupts instead
timeout polling (one of the reasons why split was made).
This is now possible with APR_WAKEABLE pollsets.
It would require rewriting Java part as well.


But how do we clean up trunk? Throw away and start from new?


Well, not sure how much stuff was added directly to trunk which
is specific to trunk only. IMO Charles added few bits, but other
then that it's just copying over from 1.1 branch.


Regards
--
^TM

-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



Re: svn commit: r1438747 - /tomcat/trunk/test/org/apache/tomcat/websocket/TestWsWebSocketContainer.java

2013-01-28 Thread Mark Thomas
On 28/01/2013 13:35, Konstantin Kolinko wrote:
> 2013/1/28 Mark Thomas :
>> On 26/01/2013 12:30, Konstantin Kolinko wrote:
>>> 2013/1/26 Mark Thomas :
 kkoli...@apache.org wrote:

> Author: kkolinko
> Date: Fri Jan 25 22:34:57 2013
> New Revision: 1438747
>
> URL: http://svn.apache.org/viewvc?rev=1438747&view=rev
> Log:
> Make the messages list synchronized as a whole, instead of just using a
> volatile reference to it.
> I am still observing random failures with TestWsWebSocketContainer, so
> an issue is not here.

 So why make the change?

>>>
>>> Because of
>>> http://svn.apache.org/viewvc?view=revision&revision=1437930
>>
>> OK. The log message seemed to suggest the change was pointless. Hence my
>> question.
>>
>>> If you just need to propagate the "messages" reference across threads,
>>> then I think r1437930 should have been more simple: just marking the
>>> field as "final".
>>> If you are protecting access to inner structures of ArrayList class, I
>>> think that marking the reference as volatile is not enough and that
>>> using a synchronized list is more adequate.
>>>
>>> I think "volatile" modifier applies to accessing the filed itself, and
>>> has no effect when you use the value returned by getMessages() or
>>> store it in a local variable.
>>>
>>> Well, I do not see where a concurrency can come from here, so mere
>>> "final" should be enough. The latch did count down (otherwise
>>> assertTrue(latchResult) detected a failure) so either a message was
>>> received or TesterEndpoint.onClose() or TesterEndpoint.onError() was
>>> called. The latch should provide synchronization between threads.
>>
>> Having just read up on "happens-before" I do not believe the list needs
>> to be synchronised.
> 
> Agreed.
> 
> I am OK with leaving it as is (as well as with CopyOnWriteArrayList
> from r1439351). It should not matter for this test case.
> 
>> I believe the root cause of the problem I was trying
>> to solve was fixed in r1437930 when I switched the order of adding the
>> message to the list and counting down the latch.
> 
> Agreed. It should be it.
> (For the record, you obviously meant r1438229 and the change in
> onMessage(..) in TestWsWebSocketContainer.java)
> 
> BTW, the test still fails for BIO in the last two buildbot runs. (NIO was OK).

Yeah, something isn't quite right yet. I need to dig into this some more.

Mark


-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



svn commit: r1439434 - /tomcat/trunk/webapps/docs/windows-auth-howto.xml

2013-01-28 Thread markt
Author: markt
Date: Mon Jan 28 14:57:07 2013
New Revision: 1439434

URL: http://svn.apache.org/viewvc?rev=1439434&view=rev
Log:
Add no-follow as suggested by kkolinko

Modified:
tomcat/trunk/webapps/docs/windows-auth-howto.xml

Modified: tomcat/trunk/webapps/docs/windows-auth-howto.xml
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/webapps/docs/windows-auth-howto.xml?rev=1439434&r1=1439433&r2=1439434&view=diff
==
--- tomcat/trunk/webapps/docs/windows-auth-howto.xml (original)
+++ tomcat/trunk/webapps/docs/windows-auth-howto.xml Mon Jan 28 14:57:07 2013
@@ -220,8 +220,8 @@ com.sun.security.jgss.krb5.accept {
 
   
   Full details of this solution can be found through the
-  http://waffle.codeplex.com/";>Waffle web site. The key features
-  are:
+  http://waffle.codeplex.com/";  rel="nofollow">Waffle web site. 
The
+  key features are:
   
   Drop-in solution
   Simple configuration (no JAAS or Kerberos keytab configuration required)
@@ -232,8 +232,8 @@ com.sun.security.jgss.krb5.accept {
 
   
   Full details of this solution can be found through the
-  http://static.springsource.org/spring-security/site/extensions/krb/index.html";>
-  Kerberos extension web site. The key features are:
+  http://static.springsource.org/spring-security/site/extensions/krb/index.html";
+  rel="nofollow"> Kerberos extension web site. The key features are:
   
   Extension to Spring Security
   Requires a Kerberos keytab file to be generated
@@ -243,8 +243,8 @@ com.sun.security.jgss.krb5.accept {
 
   
   Full details of this solution can be found through the
-  http://spnego.sourceforge.net/index.html/";>project site. The key
-  features are:
+  http://spnego.sourceforge.net/index.html/";  rel="nofollow">project
+  site. The key features are:
   
   Uses Kerberos
   Pure Java solution
@@ -253,7 +253,7 @@ com.sun.security.jgss.krb5.accept {
 
   
   Full details of this solution can be found through the
-  http://www.ioplex.com/";>project web siteThe key
+  http://www.ioplex.com/"; rel="nofollow">project web siteThe key
   features are:
   
   Pure Java solution
@@ -282,12 +282,12 @@ com.sun.security.jgss.krb5.accept {
   Apache httpd does not support Windows authentication out of the box but
   there are a number of third-party modules that can be used. These 
include:
   
-  http://sourceforge.net/projects/mod-auth-sspi/";>mod_auth_sspi
-  for use on Windows platforms.
-  http://adldap.sourceforge.net/wiki/doku.php?id=mod_auth_ntlm_winbind";>
-  mod_auth_ntlm_winbind for non-Windows platforms. Known to work with httpd
-  2.0.x on 32-bit platforms. Some users have reported stability issues with 
both
-  httpd 2.2.x builds and 64-bit Linux builds.
+  http://sourceforge.net/projects/mod-auth-sspi/";
+  rel="nofollow">mod_auth_sspi for use on Windows platforms.
+  http://adldap.sourceforge.net/wiki/doku.php?id=mod_auth_ntlm_winbind";
+  rel="nofollow">mod_auth_ntlm_winbind for non-Windows platforms. Known to
+  work with httpd 2.0.x on 32-bit platforms. Some users have reported stability
+  issues with both httpd 2.2.x builds and 64-bit Linux builds.
   
   There are three steps to configuring httpd to provide Windows
   authentication. They are:



-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



svn commit: r1439437 - in /tomcat/tc7.0.x/trunk: ./ webapps/docs/changelog.xml webapps/docs/windows-auth-howto.xml

2013-01-28 Thread markt
Author: markt
Date: Mon Jan 28 15:00:22 2013
New Revision: 1439437

URL: http://svn.apache.org/viewvc?rev=1439437&view=rev
Log:
Add Jespa
Use no-follow as suggested by kkolinko

Modified:
tomcat/tc7.0.x/trunk/   (props changed)
tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml
tomcat/tc7.0.x/trunk/webapps/docs/windows-auth-howto.xml

Propchange: tomcat/tc7.0.x/trunk/
--
  Merged /tomcat/trunk:r1439334,1439434

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=1439437&r1=1439436&r2=1439437&view=diff
==
--- tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml (original)
+++ tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml Mon Jan 28 15:00:22 2013
@@ -138,6 +138,11 @@
 54461: Improve the documentation for the compiler attribute
 in the Jasper how-to. (markt)
   
+  
+Add Jespa to the list of third-party Windows authentication providers
+and make external links in the documentation for those providers
+no-follow. (markt)
+  
 
   
   

Modified: tomcat/tc7.0.x/trunk/webapps/docs/windows-auth-howto.xml
URL: 
http://svn.apache.org/viewvc/tomcat/tc7.0.x/trunk/webapps/docs/windows-auth-howto.xml?rev=1439437&r1=1439436&r2=1439437&view=diff
==
--- tomcat/tc7.0.x/trunk/webapps/docs/windows-auth-howto.xml (original)
+++ tomcat/tc7.0.x/trunk/webapps/docs/windows-auth-howto.xml Mon Jan 28 
15:00:22 2013
@@ -51,8 +51,9 @@ sections.
 
 
 
-This is a work in progress. There are a number of outstanding
-questions that require further testing. These include:
+This documentation is a work in progress. There are a number of
+outstanding questions around the edge cases that require further
+testing. These include:
 
 
 Does the domain name have to be in upper case?
@@ -219,7 +220,8 @@ com.sun.security.jgss.krb5.accept {
 
   
   Full details of this solution can be found through the
-  http://waffle.codeplex.com/";>Waffle site. The key features 
are:
+  http://waffle.codeplex.com/";  rel="nofollow">Waffle web site. 
The
+  key features are:
   
   Drop-in solution
   Simple configuration (no JAAS or Kerberos keytab configuration required)
@@ -230,8 +232,8 @@ com.sun.security.jgss.krb5.accept {
 
   
   Full details of this solution can be found through the
-  http://static.springsource.org/spring-security/site/extensions/krb/index.html";>
-  Kerberos extension site. The key features are:
+  http://static.springsource.org/spring-security/site/extensions/krb/index.html";
+  rel="nofollow"> Kerberos extension web site. The key features are:
   
   Extension to Spring Security
   Requires a Kerberos keytab file to be generated
@@ -241,13 +243,23 @@ com.sun.security.jgss.krb5.accept {
 
   
   Full details of this solution can be found through the
-  http://spnego.sourceforge.net/index.html/";>project site. The key
-  features are:
+  http://spnego.sourceforge.net/index.html/";  rel="nofollow">project
+  site. The key features are:
   
   Uses Kerberos
   Pure Java solution
   
   
+
+  
+  Full details of this solution can be found through the
+  http://www.ioplex.com/"; rel="nofollow">project web siteThe key
+  features are:
+  
+  Pure Java solution
+  Advanced Active Directory integration
+  
+  
 
 
 
@@ -270,12 +282,12 @@ com.sun.security.jgss.krb5.accept {
   Apache httpd does not support Windows authentication out of the box but
   there are a number of third-party modules that can be used. These 
include:
   
-  http://sourceforge.net/projects/mod-auth-sspi/";>mod_auth_sspi
-  for use on Windows platforms.
-  http://adldap.sourceforge.net/wiki/doku.php?id=mod_auth_ntlm_winbind";>
-  mod_auth_ntlm_winbind for non-Windows platforms. Known to work with httpd
-  2.0.x on 32-bit platforms. Some users have reported stability issues with 
both
-  httpd 2.2.x builds and 64-bit Linux builds.
+  http://sourceforge.net/projects/mod-auth-sspi/";
+  rel="nofollow">mod_auth_sspi for use on Windows platforms.
+  http://adldap.sourceforge.net/wiki/doku.php?id=mod_auth_ntlm_winbind";
+  rel="nofollow">mod_auth_ntlm_winbind for non-Windows platforms. Known to
+  work with httpd 2.0.x on 32-bit platforms. Some users have reported stability
+  issues with both httpd 2.2.x builds and 64-bit Linux builds.
   
   There are three steps to configuring httpd to provide Windows
   authentication. They are:



-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



svn commit: r1439442 - /tomcat/trunk/java/org/apache/tomcat/util/http/parser/HttpParser.java

2013-01-28 Thread markt
Author: markt
Date: Mon Jan 28 15:07:53 2013
New Revision: 1439442

URL: http://svn.apache.org/viewvc?rev=1439442&view=rev
Log:
Follow-up to https://issues.apache.org/bugzilla/show_bug.cgi?id=54060
More buggy client implementations of DIGEST auth. This time it is the JDK 
(Oracle 6.x, Oracle 7.x, OpenJDK 7.*)

Modified:
tomcat/trunk/java/org/apache/tomcat/util/http/parser/HttpParser.java

Modified: tomcat/trunk/java/org/apache/tomcat/util/http/parser/HttpParser.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/tomcat/util/http/parser/HttpParser.java?rev=1439442&r1=1439441&r2=1439442&view=diff
==
--- tomcat/trunk/java/org/apache/tomcat/util/http/parser/HttpParser.java 
(original)
+++ tomcat/trunk/java/org/apache/tomcat/util/http/parser/HttpParser.java Mon 
Jan 28 15:07:53 2013
@@ -43,6 +43,7 @@ import java.util.Map;
  */
 public class HttpParser {
 
+@SuppressWarnings("unused")  // Unused due to buggy client implementations
 private static final Integer FIELD_TYPE_TOKEN = Integer.valueOf(0);
 private static final Integer FIELD_TYPE_QUOTED_STRING = Integer.valueOf(1);
 private static final Integer FIELD_TYPE_TOKEN_OR_QUOTED_STRING = 
Integer.valueOf(2);
@@ -63,7 +64,7 @@ public class HttpParser {
 fieldTypes.put("nonce", FIELD_TYPE_QUOTED_STRING);
 fieldTypes.put("digest-uri", FIELD_TYPE_QUOTED_STRING);
 fieldTypes.put("response", FIELD_TYPE_QUOTED_LHEX);
-fieldTypes.put("algorithm", FIELD_TYPE_TOKEN);
+fieldTypes.put("algorithm", FIELD_TYPE_QUOTED_TOKEN);
 fieldTypes.put("cnonce", FIELD_TYPE_QUOTED_STRING);
 fieldTypes.put("opaque", FIELD_TYPE_QUOTED_STRING);
 fieldTypes.put("qop", FIELD_TYPE_QUOTED_TOKEN);



-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



svn commit: r1439443 - in /tomcat/tc7.0.x/trunk: ./ java/org/apache/tomcat/util/http/parser/HttpParser.java webapps/docs/changelog.xml

2013-01-28 Thread markt
Author: markt
Date: Mon Jan 28 15:10:28 2013
New Revision: 1439443

URL: http://svn.apache.org/viewvc?rev=1439443&view=rev
Log:
Follow-up to https://issues.apache.org/bugzilla/show_bug.cgi?id=54060
More buggy client implementations of DIGEST auth. This time it is the JDK 
(Oracle 6.x, Oracle 7.x, OpenJDK 7.*)

Modified:
tomcat/tc7.0.x/trunk/   (props changed)
tomcat/tc7.0.x/trunk/java/org/apache/tomcat/util/http/parser/HttpParser.java
tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml

Propchange: tomcat/tc7.0.x/trunk/
--
  Merged /tomcat/trunk:r1439442

Modified: 
tomcat/tc7.0.x/trunk/java/org/apache/tomcat/util/http/parser/HttpParser.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc7.0.x/trunk/java/org/apache/tomcat/util/http/parser/HttpParser.java?rev=1439443&r1=1439442&r2=1439443&view=diff
==
--- 
tomcat/tc7.0.x/trunk/java/org/apache/tomcat/util/http/parser/HttpParser.java 
(original)
+++ 
tomcat/tc7.0.x/trunk/java/org/apache/tomcat/util/http/parser/HttpParser.java 
Mon Jan 28 15:10:28 2013
@@ -43,6 +43,7 @@ import java.util.Map;
  */
 public class HttpParser {
 
+@SuppressWarnings("unused")  // Unused due to buggy client implementations
 private static final Integer FIELD_TYPE_TOKEN = Integer.valueOf(0);
 private static final Integer FIELD_TYPE_QUOTED_STRING = Integer.valueOf(1);
 private static final Integer FIELD_TYPE_TOKEN_OR_QUOTED_STRING = 
Integer.valueOf(2);
@@ -64,7 +65,7 @@ public class HttpParser {
 fieldTypes.put("nonce", FIELD_TYPE_QUOTED_STRING);
 fieldTypes.put("digest-uri", FIELD_TYPE_QUOTED_STRING);
 fieldTypes.put("response", FIELD_TYPE_QUOTED_LHEX);
-fieldTypes.put("algorithm", FIELD_TYPE_TOKEN);
+fieldTypes.put("algorithm", FIELD_TYPE_QUOTED_TOKEN);
 fieldTypes.put("cnonce", FIELD_TYPE_QUOTED_STRING);
 fieldTypes.put("opaque", FIELD_TYPE_QUOTED_STRING);
 fieldTypes.put("qop", FIELD_TYPE_QUOTED_TOKEN);

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=1439443&r1=1439442&r2=1439443&view=diff
==
--- tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml (original)
+++ tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml Mon Jan 28 15:10:28 2013
@@ -57,6 +57,10 @@
   
 
   
+Make additional allowances for buggy client implementations of HTTP
+DIGEST authentication. This is a follow-on to 54060. (markt)
+  
+  
 54438: Fix a regression in the fix for 52953 that
 triggered a NPE when digested passwords were used and an authentication
 attempt was made for a user that did not exist in the realm. (markt)



-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



[Bug 54060] DigestAuthenticator doesn't parse Authorization header correctly

2013-01-28 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=54060

Mark Thomas  changed:

   What|Removed |Added

 Status|REOPENED|RESOLVED
 Resolution|--- |FIXED

--- Comment #14 from Mark Thomas  ---
Fixed in trunk and 7.0.x and will be included in 7.0.36 onwards.

algorithm is now permitted to be a quoted token. Note no such definition exists
in the specs. It is <"><"> as some clients insist on adding quotes to
tokens.

-- 
You are receiving this mail because:
You are the assignee for the bug.

-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



svn commit: r1439445 - /tomcat/trunk/java/org/apache/catalina/tribes/membership/MemberImpl.java

2013-01-28 Thread markt
Author: markt
Date: Mon Jan 28 15:18:04 2013
New Revision: 1439445

URL: http://svn.apache.org/viewvc?rev=1439445&view=rev
Log:
Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=54496
Don't use hard-coded class name in toString()

Modified:
tomcat/trunk/java/org/apache/catalina/tribes/membership/MemberImpl.java

Modified: 
tomcat/trunk/java/org/apache/catalina/tribes/membership/MemberImpl.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/tribes/membership/MemberImpl.java?rev=1439445&r1=1439444&r2=1439445&view=diff
==
--- tomcat/trunk/java/org/apache/catalina/tribes/membership/MemberImpl.java 
(original)
+++ tomcat/trunk/java/org/apache/catalina/tribes/membership/MemberImpl.java Mon 
Jan 28 15:18:04 2013
@@ -513,7 +513,8 @@ public class MemberImpl implements Membe
  */
 @Override
 public String toString()  {
-StringBuilder buf = new 
StringBuilder("org.apache.catalina.tribes.membership.MemberImpl[");
+StringBuilder buf = new StringBuilder(getClass().getName());
+buf.append(getName()).append("[");
 buf.append(getName()).append(",");
 buf.append(getHostname()).append(",");
 buf.append(port).append(", alive=");



-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



[Bug 52318] Version in tomcat-jdbc POM is conflicted with Version in MANIFEST for JULI JAR

2013-01-28 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=52318

Martin Lichtin  changed:

   What|Removed |Added

 Status|RESOLVED|REOPENED
 Resolution|LATER   |---

--- Comment #18 from Martin Lichtin  ---
Just realize there is another problem when trying to use tomcat-jdbc under
OSGi:
Juli is not a bundle. 
So the version restriction in tomcat-jdbc's MANIFEST does not really make
sense,
as Juli does not export its packages with a version.

-- 
You are receiving this mail because:
You are the assignee for the bug.

-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



svn commit: r1439447 - in /tomcat/tc7.0.x/trunk: ./ java/org/apache/catalina/tribes/membership/MemberImpl.java webapps/docs/changelog.xml

2013-01-28 Thread markt
Author: markt
Date: Mon Jan 28 15:20:35 2013
New Revision: 1439447

URL: http://svn.apache.org/viewvc?rev=1439447&view=rev
Log:
Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=54496
Don't use hard-coded class name in toString()

Modified:
tomcat/tc7.0.x/trunk/   (props changed)

tomcat/tc7.0.x/trunk/java/org/apache/catalina/tribes/membership/MemberImpl.java
tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml

Propchange: tomcat/tc7.0.x/trunk/
--
  Merged /tomcat/trunk:r1439445

Modified: 
tomcat/tc7.0.x/trunk/java/org/apache/catalina/tribes/membership/MemberImpl.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc7.0.x/trunk/java/org/apache/catalina/tribes/membership/MemberImpl.java?rev=1439447&r1=1439446&r2=1439447&view=diff
==
--- 
tomcat/tc7.0.x/trunk/java/org/apache/catalina/tribes/membership/MemberImpl.java 
(original)
+++ 
tomcat/tc7.0.x/trunk/java/org/apache/catalina/tribes/membership/MemberImpl.java 
Mon Jan 28 15:20:35 2013
@@ -534,7 +534,8 @@ public class MemberImpl implements Membe
  */
 @Override
 public String toString()  {
-StringBuilder buf = new 
StringBuilder("org.apache.catalina.tribes.membership.MemberImpl[");
+StringBuilder buf = new StringBuilder(getClass().getName());
+buf.append(getName()).append("[");
 buf.append(getName()).append(",");
 buf.append(getHostname()).append(",");
 buf.append(port).append(", alive=");

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=1439447&r1=1439446&r2=1439447&view=diff
==
--- tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml (original)
+++ tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml Mon Jan 28 15:20:35 2013
@@ -149,6 +149,14 @@
   
 
   
+  
+
+  
+54496: Don't use a hard-coded class name in
+MemberImpl.toString(). (markt)
+  
+
+  
   
 
   



-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



[Bug 54496] ..tribes.membership.MemberImpl.toString() might be better using .getClass().getName()

2013-01-28 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=54496

Mark Thomas  changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED

--- Comment #1 from Mark Thomas  ---
Thanks for the report.

This has been fixed in trunk and 7.0.x and will be included in 7.0.36 onwards.

-- 
You are receiving this mail because:
You are the assignee for the bug.

-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



buildbot success in ASF Buildbot on tomcat-trunk

2013-01-28 Thread buildbot
The Buildbot has detected a restored build on builder tomcat-trunk while 
building ASF Buildbot.
Full details are available at:
 http://ci.apache.org/builders/tomcat-trunk/builds/3848

Buildbot URL: http://ci.apache.org/

Buildslave for this Build: bb-vm_ubuntu

Build Reason: scheduler
Build Source Stamp: [branch tomcat/trunk] 1439434
Blamelist: markt

Build succeeded!

sincerely,
 -The Buildbot





Re: Confusion around tcnative

2013-01-28 Thread Christopher Schultz
Rainer,

On 1/28/13 6:20 AM, Rainer Jung wrote:
> I'm a bit confused about tcnative and the various trunks and branches:
> 
> 1) tcnative trunk vs. branch 1.1.x.
> 
> It seems most updates are directly done to the branch. So trunk is
> outdated. On the other hand originally there were some updates to trunk
> not in 1.1.x. I think this should be cleaned up. The current trunk
> doesn't seem to be useful.

I've tried to always commit to trunk and back-port to 1.1.x. I have had
some irritation building trunk because it requires bleeding-edge
versions of APR that I don't have available from Debian. It's a minor
inconvenience, honestly.

trunk 'configure' also does not properly support --with-java or
--with-java-home like 1.1.x does. That would be helpful to fix... my
configure-fu is not strong.

> 2) tcnative and Java classes
> 
> We still have Java classes in the tcnative branch. They neither match
> with the copies maintained in the tc7 branch, nor with the ones in the
> tc trunk branch. tcnative trunk does not have the Java files.

+1 to removing the Java sources from tcnative in 1.1.x branch. We have
had at least one recent report on the users' list where someone
hand-compiled the Java sources and was then unable to use tcnative
successfully (because the Java sources were out of date with respect to
those in the main Tomcat sources).

> Is there any use for the copies of the Java files in the tcnative 1.1.x
> branch? Otherwise I think we should remov them and check them out from
> TC7 or TC trunk when rolling a tcnative release.

I'm not even sure that a tcnative release requires any .class or .jar
files, as Tomcat itself contains everything necessary. I believe they
are tightly-coupled enough with the rest of Tomcat that shipping a JAR
file along with the tcnative library might break installations with
mismatched Java code accompanying the native stuff.

> Note that tcnative also contains test and examples Java classes, which
> are *not* also in the TC branches. Those must be kept in tcnative (or
> moved to TC).

+1 to moving/adapting to Tomcat.

-chris



signature.asc
Description: OpenPGP digital signature


svn commit: r1439667 - /tomcat/trunk/java/org/apache/catalina/tribes/membership/MemberImpl.java

2013-01-28 Thread kkolinko
Author: kkolinko
Date: Mon Jan 28 22:06:56 2013
New Revision: 1439667

URL: http://svn.apache.org/viewvc?rev=1439667&view=rev
Log:
Correct a glitch in r1439445.
Remove duplicate "append(getName())" that was copied from the next line.

Modified:
tomcat/trunk/java/org/apache/catalina/tribes/membership/MemberImpl.java

Modified: 
tomcat/trunk/java/org/apache/catalina/tribes/membership/MemberImpl.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/tribes/membership/MemberImpl.java?rev=1439667&r1=1439666&r2=1439667&view=diff
==
--- tomcat/trunk/java/org/apache/catalina/tribes/membership/MemberImpl.java 
(original)
+++ tomcat/trunk/java/org/apache/catalina/tribes/membership/MemberImpl.java Mon 
Jan 28 22:06:56 2013
@@ -514,7 +514,7 @@ public class MemberImpl implements Membe
 @Override
 public String toString()  {
 StringBuilder buf = new StringBuilder(getClass().getName());
-buf.append(getName()).append("[");
+buf.append("[");
 buf.append(getName()).append(",");
 buf.append(getHostname()).append(",");
 buf.append(port).append(", alive=");



-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



svn commit: r1439669 - in /tomcat/tc7.0.x/trunk: ./ java/org/apache/catalina/tribes/membership/MemberImpl.java

2013-01-28 Thread kkolinko
Author: kkolinko
Date: Mon Jan 28 22:10:44 2013
New Revision: 1439669

URL: http://svn.apache.org/viewvc?rev=1439669&view=rev
Log:
Merged revision 1439667 from tomcat/trunk:
Correct the fix for https://issues.apache.org/bugzilla/show_bug.cgi?id=54496
Remove duplicate "append(getName())" that was copied from the next line.

Modified:
tomcat/tc7.0.x/trunk/   (props changed)

tomcat/tc7.0.x/trunk/java/org/apache/catalina/tribes/membership/MemberImpl.java

Propchange: tomcat/tc7.0.x/trunk/
--
  Merged /tomcat/trunk:r1439667

Modified: 
tomcat/tc7.0.x/trunk/java/org/apache/catalina/tribes/membership/MemberImpl.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc7.0.x/trunk/java/org/apache/catalina/tribes/membership/MemberImpl.java?rev=1439669&r1=1439668&r2=1439669&view=diff
==
--- 
tomcat/tc7.0.x/trunk/java/org/apache/catalina/tribes/membership/MemberImpl.java 
(original)
+++ 
tomcat/tc7.0.x/trunk/java/org/apache/catalina/tribes/membership/MemberImpl.java 
Mon Jan 28 22:10:44 2013
@@ -535,7 +535,7 @@ public class MemberImpl implements Membe
 @Override
 public String toString()  {
 StringBuilder buf = new StringBuilder(getClass().getName());
-buf.append(getName()).append("[");
+buf.append("[");
 buf.append(getName()).append(",");
 buf.append(getHostname()).append(",");
 buf.append(port).append(", alive=");



-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



svn commit: r1439678 - /tomcat/trunk/java/org/apache/tomcat/websocket/WsFrameBase.java

2013-01-28 Thread markt
Author: markt
Date: Mon Jan 28 22:42:09 2013
New Revision: 1439678

URL: http://svn.apache.org/viewvc?rev=1439678&view=rev
Log:
Handle session close higher up the call stack

Modified:
tomcat/trunk/java/org/apache/tomcat/websocket/WsFrameBase.java

Modified: tomcat/trunk/java/org/apache/tomcat/websocket/WsFrameBase.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/tomcat/websocket/WsFrameBase.java?rev=1439678&r1=1439677&r2=1439678&view=diff
==
--- tomcat/trunk/java/org/apache/tomcat/websocket/WsFrameBase.java (original)
+++ tomcat/trunk/java/org/apache/tomcat/websocket/WsFrameBase.java Mon Jan 28 
22:42:09 2013
@@ -497,8 +497,7 @@ public abstract class WsFrameBase {
 sm.getString("wsFrame.bufferToSmall",
 Integer.valueOf(inputBuffer.length),
 Long.valueOf(payloadLength)));
-wsSession.close(cr);
-throw new IOException(cr.getReasonPhrase());
+throw new WsIOException(cr);
 }
 makeRoom();
 }



-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



svn commit: r1439679 - /tomcat/trunk/java/org/apache/tomcat/websocket/WsFrameClient.java

2013-01-28 Thread markt
Author: markt
Date: Mon Jan 28 22:42:48 2013
New Revision: 1439679

URL: http://svn.apache.org/viewvc?rev=1439679&view=rev
Log:
Look to see if a CloseReason is available before generating one.

Modified:
tomcat/trunk/java/org/apache/tomcat/websocket/WsFrameClient.java

Modified: tomcat/trunk/java/org/apache/tomcat/websocket/WsFrameClient.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/tomcat/websocket/WsFrameClient.java?rev=1439679&r1=1439678&r2=1439679&view=diff
==
--- tomcat/trunk/java/org/apache/tomcat/websocket/WsFrameClient.java (original)
+++ tomcat/trunk/java/org/apache/tomcat/websocket/WsFrameClient.java Mon Jan 28 
22:42:48 2013
@@ -68,8 +68,14 @@ public class WsFrameClient extends WsFra
 
 
 private final void close(Throwable t) {
-CloseReason cr = new CloseReason(
+CloseReason cr;
+if (t instanceof WsIOException) {
+cr = ((WsIOException) t).getCloseReason();
+} else {
+cr = new CloseReason(
 CloseCodes.CLOSED_ABNORMALLY, t.getMessage());
+}
+
 try {
 wsSession.close(cr);
 } catch (IOException ignore) {



-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



buildbot failure in ASF Buildbot on tomcat-7-trunk

2013-01-28 Thread buildbot
The Buildbot has detected a new failure on builder tomcat-7-trunk while 
building ASF Buildbot.
Full details are available at:
 http://ci.apache.org/builders/tomcat-7-trunk/builds/1044

Buildbot URL: http://ci.apache.org/

Buildslave for this Build: bb-vm_ubuntu

Build Reason: scheduler
Build Source Stamp: [branch tomcat/tc7.0.x/trunk] 1439669
Blamelist: kkolinko

BUILD FAILED: failed compile_1

sincerely,
 -The Buildbot





buildbot failure in ASF Buildbot on tomcat-trunk

2013-01-28 Thread buildbot
The Buildbot has detected a new failure on builder tomcat-trunk while building 
ASF Buildbot.
Full details are available at:
 http://ci.apache.org/builders/tomcat-trunk/builds/3851

Buildbot URL: http://ci.apache.org/

Buildslave for this Build: bb-vm_ubuntu

Build Reason: scheduler
Build Source Stamp: [branch tomcat/trunk] 1439679
Blamelist: markt

BUILD FAILED: failed compile_1

sincerely,
 -The Buildbot





svn commit: r1439757 - /tomcat/trunk/java/org/apache/catalina/authenticator/SingleSignOn.java

2013-01-28 Thread kfujino
Author: kfujino
Date: Tue Jan 29 07:04:52 2013
New Revision: 1439757

URL: http://svn.apache.org/viewvc?rev=1439757&view=rev
Log:
Prevent the SSO deregister when web application is stopped or reloaded.
When StandardManager(pathname="") or DeltaManager stops normally, all sessions 
in the context are expired.
In this case, because most sessions is not time-out, SSO deregister was 
triggered.

Modified:
tomcat/trunk/java/org/apache/catalina/authenticator/SingleSignOn.java

Modified: tomcat/trunk/java/org/apache/catalina/authenticator/SingleSignOn.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/authenticator/SingleSignOn.java?rev=1439757&r1=1439756&r2=1439757&view=diff
==
--- tomcat/trunk/java/org/apache/catalina/authenticator/SingleSignOn.java 
(original)
+++ tomcat/trunk/java/org/apache/catalina/authenticator/SingleSignOn.java Tue 
Jan 29 07:04:52 2013
@@ -224,7 +224,8 @@ public class SingleSignOn extends ValveB
 if (((session.getMaxInactiveInterval() > 0)
 && (System.currentTimeMillis() - 
session.getThisAccessedTimeInternal() >=
 session.getMaxInactiveInterval() * 1000))
-|| (Session.SESSION_PASSIVATED_EVENT.equals(event.getType( {
+|| (Session.SESSION_PASSIVATED_EVENT.equals(event.getType()))
+|| (!session.getManager().getContext().getState().isAvailable())) {
 removeSession(ssoId, session);
 } else {
 // The session was logged out.



-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



svn commit: r1439758 - in /tomcat/tc7.0.x/trunk: java/org/apache/catalina/authenticator/SingleSignOn.java webapps/docs/changelog.xml

2013-01-28 Thread kfujino
Author: kfujino
Date: Tue Jan 29 07:08:16 2013
New Revision: 1439758

URL: http://svn.apache.org/viewvc?rev=1439758&view=rev
Log:
Prevent the SSO deregister when web application is stopped or reloaded.
When StandardManager(pathname="") or DeltaManager stops normally, all sessions 
in the context are expired.
In this case, because most sessions is not time-out, SSO deregister was 
triggered.

Modified:

tomcat/tc7.0.x/trunk/java/org/apache/catalina/authenticator/SingleSignOn.java
tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml

Modified: 
tomcat/tc7.0.x/trunk/java/org/apache/catalina/authenticator/SingleSignOn.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc7.0.x/trunk/java/org/apache/catalina/authenticator/SingleSignOn.java?rev=1439758&r1=1439757&r2=1439758&view=diff
==
--- 
tomcat/tc7.0.x/trunk/java/org/apache/catalina/authenticator/SingleSignOn.java 
(original)
+++ 
tomcat/tc7.0.x/trunk/java/org/apache/catalina/authenticator/SingleSignOn.java 
Tue Jan 29 07:08:16 2013
@@ -238,7 +238,8 @@ public class SingleSignOn extends ValveB
 if (((session.getMaxInactiveInterval() > 0)
 && (System.currentTimeMillis() - 
session.getThisAccessedTimeInternal() >=
 session.getMaxInactiveInterval() * 1000)) 
-|| (Session.SESSION_PASSIVATED_EVENT.equals(event.getType( {
+|| (Session.SESSION_PASSIVATED_EVENT.equals(event.getType()))
+|| 
(!session.getManager().getContainer().getState().isAvailable())) {
 removeSession(ssoId, session);
 } else {
 // The session was logged out.

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=1439758&r1=1439757&r2=1439758&view=diff
==
--- tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml (original)
+++ tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml Tue Jan 29 07:08:16 2013
@@ -83,6 +83,13 @@
 54483: Correct one of the Spanish translations. Based on a
 suggestion from adinamita. (markt)
   
+  
+Prevent the SSO deregister when web application is stopped or reloaded.
+When StandardManager(pathname="") or DeltaManager stops normally, all
+sessions in the context are expired.
+In this case, because most sessions is not time-out, SSO deregister was
+triggered. (kfujino)
+  
 
   
   



-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org