Re: DO NOT REPLY [Bug 47767] [PatchAvailable] mod_actions and Server-Status

2009-09-01 Thread sebb
The same person also raised a JIRA issue.
[It was INFRA-2178, now deleted.]



On 31/08/2009, Mark Thomas  wrote:
> This looks like spam to me.
>
>  I am going to purge it from Bugzilla and disable the account of the use
>  that created it.
>
>
>  Mark
>
>
>
>
>  -
>  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



DO NOT REPLY [Bug 47774] New: Illegal context class loader is used when HttpSessionListener is executed.

2009-09-01 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=47774

   Summary: Illegal context class loader is used when
HttpSessionListener is executed.
   Product: Tomcat 6
   Version: 6.0.20
  Platform: All
OS/Version: All
Status: NEW
  Severity: normal
  Priority: P2
 Component: Catalina
AssignedTo: dev@tomcat.apache.org
ReportedBy: fujino.keii...@oss.ntt.co.jp


--- Comment #0 from Keiichi Fujino  2009-09-01 
01:58:42 PDT ---
When HttpSessionListener is executed, illegal class loader is set to the
context classLoader.

I think that the problem is in CoyoteAdapter#parseSessionCookiesId method. 
JSESSION COOKIE is parsed in this method, and sessionId is set to the request. 

To check sessionId, Request#isRequestedSessionIdValid method is called in this
method. 
And, Session#isValid() might be called in Request#isRequestedSessionIdValid
method. 
However, the context class loader of a current thread is StandardClassLoader. 
It is not WebappClassLoader. 

For instance,
When the session has already passed session-timeout, 
Session#expire is executed.
At this time, the context class loader of the thread that executes
HttpSessionListenner#sessionDestroyed is StanderdClassLoader. 
This is not good. 
The context class loader of the thread that executes HttpSessionListenner
should be WebAppClassLoader. 


Best regards.

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- 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: JSR 315 - status ?

2009-09-01 Thread Matthias Wessendorf
Is there no Tomcat developer an active member on the JSR ?

-Matthias

On Mon, Aug 31, 2009 at 8:51 PM, Matthias Wessendorf wrote:
> Hi there,
>
> Does one know what the current status of JSR 315 is ? Will it be final soon?
> I think last what I heard was that Filip said - on ACon EU, in April,
> that it may
> be final in September 09.
>
> -Matthias
>
> --
> Matthias Wessendorf
>
> blog: http://matthiaswessendorf.wordpress.com/
> sessions: http://www.slideshare.net/mwessendorf
> twitter: http://twitter.com/mwessendorf
>



-- 
Matthias Wessendorf

blog: http://matthiaswessendorf.wordpress.com/
sessions: http://www.slideshare.net/mwessendorf
twitter: http://twitter.com/mwessendorf

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



Re: JSR 315 - status ?

2009-09-01 Thread Remy Maucherat
On Tue, 2009-09-01 at 13:40 +0200, Matthias Wessendorf wrote:
> Is there no Tomcat developer an active member on the JSR ?

You do understand this is confidential information, right ? Follow the
JCP progress and announcements.

Rémy



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



Re: JSR 315 - status ?

2009-09-01 Thread Matthias Wessendorf
On Tue, Sep 1, 2009 at 1:47 PM, Remy Maucherat wrote:
> On Tue, 2009-09-01 at 13:40 +0200, Matthias Wessendorf wrote:
>> Is there no Tomcat developer an active member on the JSR ?
>
> You do understand this is confidential information, right ? Follow the
> JCP progress and announcements.

ah, yeah the process. OK, is there a t...@tomcat.a.o list or similar ?
At Apache MyFaces we have that. I signed already the NDA agreement
for TCKs etc. So, I am still interested in the ETA of JSR 315. Any way
to get to this kind of information ?

Thanks,
Matthias Wessendorf
-PMC Chair Apache MyFaces-

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



-- 
Matthias Wessendorf

blog: http://matthiaswessendorf.wordpress.com/
sessions: http://www.slideshare.net/mwessendorf
twitter: http://twitter.com/mwessendorf

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



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

2009-09-01 Thread fhanik
Author: fhanik
Date: Tue Sep  1 17:21:59 2009
New Revision: 810132

URL: http://svn.apache.org/viewvc?rev=810132&view=rev
Log:
proposal

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=810132&r1=810131&r2=810132&view=diff
==
--- tomcat/tc6.0.x/trunk/STATUS.txt (original)
+++ tomcat/tc6.0.x/trunk/STATUS.txt Tue Sep  1 17:21:59 2009
@@ -321,3 +321,25 @@
   +1: funkman
   -1:
 
+
+* Fix VirtualWebappClassLoader file directives
+  Index: java/org/apache/catalina/loader/VirtualWebappLoader.java
+===
+--- java/org/apache/catalina/loader/VirtualWebappLoader.java   (revision 
810099)
 java/org/apache/catalina/loader/VirtualWebappLoader.java   (working copy)
+@@ -92,9 +92,9 @@
+ continue;
+ }
+ if (file.isDirectory()) {
+-addRepository("file:/" + file.getAbsolutePath() + "/");
++addRepository("file://" + file.getAbsolutePath() + "/");
+ } else {
+-addRepository("file:/" + file.getAbsolutePath());
++addRepository("file://" + file.getAbsolutePath());
+ }
+ }
+
+  +1: fhanik
+  -1: 
+
+



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



DO NOT REPLY [Bug 47750] Loss of worker settings when changing via jkstatus

2009-09-01 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=47750



--- Comment #1 from Rainer Jung  2009-09-01 10:50:22 
PDT ---
I can see the problem. In order to fix it in the right way, I would like to
understand, why the redirector does a second initialization.

As far as I can see, it is a second IIS process (separate process ID) that
attaches to the shm and wipes it out. I can easily prevent that efect (the
zeroing of the shared memory), but it would be more correct, if the second
process actually got the right data from the already existing shared memory.

Are you using web gardens and/or application pools? Which one of those and how
are they configured?

Could you please also provide your workers.properties and
uriworkermap.properties as well as the isapi_redirect.properties.

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- 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: r810161 - /tomcat/tc6.0.x/trunk/STATUS.txt

2009-09-01 Thread rjung
Author: rjung
Date: Tue Sep  1 18:40:32 2009
New Revision: 810161

URL: http://svn.apache.org/viewvc?rev=810161&view=rev
Log:
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=810161&r1=810160&r2=810161&view=diff
==
--- tomcat/tc6.0.x/trunk/STATUS.txt (original)
+++ tomcat/tc6.0.x/trunk/STATUS.txt Tue Sep  1 18:40:32 2009
@@ -174,7 +174,7 @@
 
 * Fix NPE observed when unpackWARs="false" with above fix
   http://svn.apache.org/viewvc?view=rev&revision=805153
-  +1: markt, funkman
+  +1: markt, funkman, rjung
   -1: 
 
 * Port Active Directory improvements to JNDIREalm from trunk
@@ -237,14 +237,17 @@
 * Expose filter configuration via JMX (read-only)
   Based on a patch by Xie Xiaodong as part of GSOC2009.
   http://svn.apache.org/viewvc?rev=802727&view=rev
-  +1: markt, funkman
+  +1: markt, funkman, rjung
   -1: 
+  rjung: typo regsiter -> register should be corrected everywhere (method name,
+ log messages etc., string manager property, sometimes with 
capitalzation "Regsiter")
+ Another little style nitpick: "if(" -> "if (".
 
 * Use platform MBean server rather than creating a new one. Makes it possible 
to
   connect JConsole and other JMX tools to locally running Tomcat instances
   without any config changes to Tomcat
   http://svn.apache.org/viewvc?rev=677910&view=rev 
-  +1: markt, funkman
+  +1: markt, funkman, rjung
   -1: 
 
 * Add remote JMX listener. Enables JMX to be configured so that it can be used
@@ -252,8 +255,12 @@
   uses one fixed and one random port.
   Needs the platform MBean server fix above to be really useful
   http://people.apache.org/~markt/patches/2009-08-12-jmx-remote.patch
-  +1: markt, funkman
+  +1: markt, funkman, rjung
   -1: 
+  rjung: The docs page contains a typo: rmiRegsitryPortPlatform -> 
rmiRegistryPortPlatform
+ There are also some hidden defaults (password and access file) which 
might make
+ sense to be added to the docs. Directory for those default files is
+ CATALINA_BASE and not CATALINA_BASE/conf?
 
 * Make context deployment error message more meaningful
   http://svn.apache.org/viewvc?rev=802940&view=rev
@@ -264,7 +271,7 @@
   https://issues.apache.org/bugzilla/show_bug.cgi?id=47046
   Clean up Jk components that only exist in the 6.0.x and were hence missed
   https://issues.apache.org/bugzilla/attachment.cgi?id=24125
-  +1: markt, funkman
+  +1: markt, funkman, rjung
   -1: 
 
 * Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=45403
@@ -318,7 +325,7 @@
 * Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=47705
   ArithmeticException: / by zero when expiring sessions via manager 
   http://svn.apache.org/viewvc?rev=809603&view=rev
-  +1: funkman
+  +1: funkman, rjung
   -1:
 
 
@@ -339,7 +346,5 @@
  }
  }
 
-  +1: fhanik
+  +1: fhanik, rjung
   -1: 
-
-



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



svn commit: r810164 - /tomcat/tc5.5.x/trunk/STATUS.txt

2009-09-01 Thread rjung
Author: rjung
Date: Tue Sep  1 18:44:26 2009
New Revision: 810164

URL: http://svn.apache.org/viewvc?rev=810164&view=rev
Log:
Vote

Modified:
tomcat/tc5.5.x/trunk/STATUS.txt

Modified: tomcat/tc5.5.x/trunk/STATUS.txt
URL: 
http://svn.apache.org/viewvc/tomcat/tc5.5.x/trunk/STATUS.txt?rev=810164&r1=810163&r2=810164&view=diff
==
--- tomcat/tc5.5.x/trunk/STATUS.txt (original)
+++ tomcat/tc5.5.x/trunk/STATUS.txt Tue Sep  1 18:44:26 2009
@@ -87,7 +87,7 @@
 
 * Fix a couple of Java 1.5 uses that should be 1.4 compatible
   http://people.apache.org/~markt/patches/2009-08-12-tc5.5.x-java14.patch
-  +1: markt
+  +1: markt, rjung
   -1: 
 
 * Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=42390



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



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

2009-09-01 Thread Larry Isaacs
Hi Filip,

I have a vague recollection about tripping over this issue while working on the 
Tomcat support in the Eclipse Web Tools Platform.  WTP has its own version(s) 
of VirtualWebappLoader, called WtpWebappLoader, to address it.  If I'm 
recalling the issue correctly, pre-pending "file:/" worked on Windows, but not 
on Linux since an absolute path ended up as "file://somedir/...".  I believe 
with the change below, pre-pending "file://" will work for Linux, but not for 
Windows, i.e. "file://C:/somedir/..." is going to have problems.  If it helps, 
I went with the URL form "file:/somedir/..." and used the following code 
snippet to get it working for Windows and Linux in WtpWebappLoader:


String path = file.getAbsolutePath();
if (path.startsWith("/")) {
path = "file:" + path;
} else {
path = "file:/" + path;
}
if (file.isDirectory()) {
addRepository(path + "/");
} else {
addRepository(path);
}

Cheers,
Larry


> -Original Message-
> From: fha...@apache.org [mailto:fha...@apache.org]
> Sent: Tuesday, September 01, 2009 1:22 PM
> To: dev@tomcat.apache.org
> Subject: svn commit: r810132 - /tomcat/tc6.0.x/trunk/STATUS.txt
>
> Author: fhanik
> Date: Tue Sep  1 17:21:59 2009
> New Revision: 810132
>
> URL: http://svn.apache.org/viewvc?rev=810132&view=rev
> Log:
> proposal
>
> 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=810132
> &r1=810131&r2=810132&view=diff
> ===
> ===
> --- tomcat/tc6.0.x/trunk/STATUS.txt (original)
> +++ tomcat/tc6.0.x/trunk/STATUS.txt Tue Sep  1 17:21:59 2009
> @@ -321,3 +321,25 @@
>+1: funkman
>-1:
>
> +
> +* Fix VirtualWebappClassLoader file directives
> +  Index: java/org/apache/catalina/loader/VirtualWebappLoader.java
> +===
> +--- java/org/apache/catalina/loader/VirtualWebappLoader.java
>   (revision 810099)
>  java/org/apache/catalina/loader/VirtualWebappLoader.java
>   (working copy)
> +@@ -92,9 +92,9 @@
> + continue;
> + }
> + if (file.isDirectory()) {
> +-addRepository("file:/" + file.getAbsolutePath() +
> "/");
> ++addRepository("file://" + file.getAbsolutePath() +
> "/");
> + } else {
> +-addRepository("file:/" + file.getAbsolutePath());
> ++addRepository("file://" + file.getAbsolutePath());
> + }
> + }
> +
> +  +1: fhanik
> +  -1:
> +
> +
>
>
>
> -
> To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
> For additional commands, e-mail: dev-h...@tomcat.apache.org
>



DO NOT REPLY [Bug 47750] Loss of worker settings when changing via jkstatus

2009-09-01 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=47750



--- Comment #2 from robert.ma...@capita.co.uk 2009-09-01 12:06:42 PDT ---
Created an attachment (id=24198)
workers.properties file

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- 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



DO NOT REPLY [Bug 47750] Loss of worker settings when changing via jkstatus

2009-09-01 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=47750



--- Comment #3 from robert.ma...@capita.co.uk 2009-09-01 12:07:06 PDT ---
Created an attachment (id=24199)
isapi redirect file

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- 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: r810176 - in /tomcat/tc5.5.x/trunk: STATUS.txt build/build.xml build/resources/build.xml

2009-09-01 Thread rjung
Author: rjung
Date: Tue Sep  1 19:07:21 2009
New Revision: 810176

URL: http://svn.apache.org/viewvc?rev=810176&view=rev
Log:
Fix build.xml comment. Backport from trunk.

Modified:
tomcat/tc5.5.x/trunk/STATUS.txt
tomcat/tc5.5.x/trunk/build/build.xml
tomcat/tc5.5.x/trunk/build/resources/build.xml

Modified: tomcat/tc5.5.x/trunk/STATUS.txt
URL: 
http://svn.apache.org/viewvc/tomcat/tc5.5.x/trunk/STATUS.txt?rev=810176&r1=810175&r2=810176&view=diff
==
--- tomcat/tc5.5.x/trunk/STATUS.txt (original)
+++ tomcat/tc5.5.x/trunk/STATUS.txt Tue Sep  1 19:07:21 2009
@@ -67,24 +67,6 @@
   +1: kkolinko
   -1:
 
-* Improve build.properties comment in ant build files.
-  Backport from trunk, only the part on build.xml.
-  http://svn.apache.org/viewvc?rev=797425&view=rev
-  +1: rjung, kkolinko
-  -1:
-  kkolinko: (
-1. Comments are documentation, thus CTR is fine for them.
-2. The phrase "See "build.properties.default" in the top level directory"
-is a bit wrong here: here it is in the build directory, not in the top
-level one.
-
-3. There are two other files besides build/build.xml in 5.5:
- a) build/resources/build.xml that is copied
- the top level directory of sources archive at build time. See *-src.zip
- file from a release.
- b) webapps/docs/build.xml It is mentioned in the docs (in building.html).
-  )
-
 * Fix a couple of Java 1.5 uses that should be 1.4 compatible
   http://people.apache.org/~markt/patches/2009-08-12-tc5.5.x-java14.patch
   +1: markt, rjung

Modified: tomcat/tc5.5.x/trunk/build/build.xml
URL: 
http://svn.apache.org/viewvc/tomcat/tc5.5.x/trunk/build/build.xml?rev=810176&r1=810175&r2=810176&view=diff
==
--- tomcat/tc5.5.x/trunk/build/build.xml (original)
+++ tomcat/tc5.5.x/trunk/build/build.xml Tue Sep  1 19:07:21 2009
@@ -20,8 +20,15 @@
 
   
 
-  
-  
+  
+  
+  
+  
+  
+  
+  
+  
+  
   
   
 

Modified: tomcat/tc5.5.x/trunk/build/resources/build.xml
URL: 
http://svn.apache.org/viewvc/tomcat/tc5.5.x/trunk/build/resources/build.xml?rev=810176&r1=810175&r2=810176&view=diff
==
--- tomcat/tc5.5.x/trunk/build/resources/build.xml (original)
+++ tomcat/tc5.5.x/trunk/build/resources/build.xml Tue Sep  1 19:07:21 2009
@@ -21,8 +21,15 @@
 
   
 
-  
-  
+  
+  
+  
+  
+  
+  
+  
+  
+  
   
   
   



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



DO NOT REPLY [Bug 47750] Loss of worker settings when changing via jkstatus

2009-09-01 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=47750



--- Comment #4 from robert.ma...@capita.co.uk 2009-09-01 12:07:28 PDT ---
Created an attachment (id=24200)
URI worker map

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- 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



DO NOT REPLY [Bug 47750] Loss of worker settings when changing via jkstatus

2009-09-01 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=47750



--- Comment #5 from robert.ma...@capita.co.uk 2009-09-01 12:15:31 PDT ---
(In reply to comment #1)
> I can easily prevent that efect (the
> zeroing of the shared memory), but it would be more correct, if the second
> process actually got the right data from the already existing shared memory.
I did try zeroing the data to start with, but yes, it's not ideal for the
worker status/changes to not be persistent when the second process starts!

> Are you using web gardens and/or application pools? Which one of those and how
> are they configured?
Just running it out of the DefaultAppPool.
Recycle worker processes set to 1740 minutes; Shutdown idle worker processes
after being idle for 20 minutes; Limit kernel requests to 100; Maximum number
of worker processes in the webgarden is set to 1.
Pinging is enabled and set to 30 seconds.
Rapid-fail protection is set to 5 failures in 5 minutes.
Startup and shutdown time limits are set to 90 seconds.
Process is running as Network Service.

There are other applications running in the DefaultAppPool - I'm not sure if
these could be influencing the second process starting?

> Could you please also provide your workers.properties and
> uriworkermap.properties as well as the isapi_redirect.properties.
Have attached these to the bug report.

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- 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: svn commit: r810132 - /tomcat/tc6.0.x/trunk/STATUS.txt

2009-09-01 Thread Rainer Jung
On 01.09.2009 20:48, Larry Isaacs wrote:
> Hi Filip,
> 
> I have a vague recollection about tripping over this issue while working on 
> the Tomcat support in the Eclipse Web Tools Platform.  WTP has its own 
> version(s) of VirtualWebappLoader, called WtpWebappLoader, to address it.  If 
> I'm recalling the issue correctly, pre-pending "file:/" worked on Windows, 
> but not on Linux since an absolute path ended up as "file://somedir/...".  I 
> believe with the change below, pre-pending "file://" will work for Linux, but 
> not for Windows, i.e. "file://C:/somedir/..." is going to have problems.  If 
> it helps, I went with the URL form "file:/somedir/..." and used the following 
> code snippet to get it working for Windows and Linux in WtpWebappLoader:

Good point.

> 
> String path = file.getAbsolutePath();
> if (path.startsWith("/")) {
> path = "file:" + path;
> } else {
> path = "file:/" + path;
> }
> if (file.isDirectory()) {
> addRepository(path + "/");
> } else {
> addRepository(path);

Why not: file.toURI().toString()

It seems this is not cheap, but should handle as much annoyances as
possible (drive letters, directory trailing slashes, UNC notation).

Regards,

Rainer

>> -Original Message-
>> From: fha...@apache.org [mailto:fha...@apache.org]
>> Sent: Tuesday, September 01, 2009 1:22 PM
>> To: dev@tomcat.apache.org
>> Subject: svn commit: r810132 - /tomcat/tc6.0.x/trunk/STATUS.txt
>>
>> Author: fhanik
>> Date: Tue Sep  1 17:21:59 2009
>> New Revision: 810132
>>
>> URL: http://svn.apache.org/viewvc?rev=810132&view=rev
>> Log:
>> proposal
>>
>> 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=810132
>> &r1=810131&r2=810132&view=diff
>> ===
>> ===
>> --- tomcat/tc6.0.x/trunk/STATUS.txt (original)
>> +++ tomcat/tc6.0.x/trunk/STATUS.txt Tue Sep  1 17:21:59 2009
>> @@ -321,3 +321,25 @@
>>+1: funkman
>>-1:
>>
>> +
>> +* Fix VirtualWebappClassLoader file directives
>> +  Index: java/org/apache/catalina/loader/VirtualWebappLoader.java
>> +===
>> +--- java/org/apache/catalina/loader/VirtualWebappLoader.java
>>   (revision 810099)
>>  java/org/apache/catalina/loader/VirtualWebappLoader.java
>>   (working copy)
>> +@@ -92,9 +92,9 @@
>> + continue;
>> + }
>> + if (file.isDirectory()) {
>> +-addRepository("file:/" + file.getAbsolutePath() +
>> "/");
>> ++addRepository("file://" + file.getAbsolutePath() +
>> "/");
>> + } else {
>> +-addRepository("file:/" + file.getAbsolutePath());
>> ++addRepository("file://" + file.getAbsolutePath());
>> + }
>> + }
>> +
>> +  +1: fhanik
>> +  -1:
>> +
>> +
>>
>>
>>
>> -
>> To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
>> For additional commands, e-mail: dev-h...@tomcat.apache.org
>>
> 
> 

-- 
kippdata
informationstechnologie GmbH   Tel: 0228 98549 -0
Bornheimer Str. 33aFax: 0228 98549 -50
53111 Bonn www.kippdata.de

HRB 8018 Amtsgericht Bonn / USt.-IdNr. DE 196 457 417
Geschäftsführer: Dr. Thomas Höfer, Rainer Jung, Sven Maurmann
===
kippdata
informationstechnologie GmbH   Tel: +49 228 98549 -0
Bornheimer Str. 33aFax: +49 228 98549 -50
D-53111 Bonn   www.kippdata.de

HRB 8018 Amtsgericht Bonn / USt.-IdNr. DE 196 457 417
Geschäftsführer: Dr. Thomas Höfer, Rainer Jung, Sven Maurmann


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



DO NOT REPLY [Bug 47750] Loss of worker settings when changing via jkstatus

2009-09-01 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=47750



--- Comment #6 from Rainer Jung  2009-09-01 12:41:20 
PDT ---
Thanks for the config, will have a look.

The request which triggered the second start was using the URL

/EODPut/225/TEST.EOD225.20090826.txt

and going to the same server name as all the other requests. Still reasoning,
why this request started another (second) process.

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- 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



DO NOT REPLY [Bug 47750] Loss of worker settings when changing via jkstatus

2009-09-01 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=47750



--- Comment #7 from Rainer Jung  2009-09-01 12:43:14 
PDT ---
config looks resonable.

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- 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: r810193 - /tomcat/tc6.0.x/trunk/STATUS.txt

2009-09-01 Thread rjung
Author: rjung
Date: Tue Sep  1 19:44:20 2009
New Revision: 810193

URL: http://svn.apache.org/viewvc?rev=810193&view=rev
Log:
Change 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=810193&r1=810192&r2=810193&view=diff
==
--- tomcat/tc6.0.x/trunk/STATUS.txt (original)
+++ tomcat/tc6.0.x/trunk/STATUS.txt Tue Sep  1 19:44:20 2009
@@ -346,5 +346,6 @@
  }
  }
 
-  +1: fhanik, rjung
+  +1: fhanik
   -1: 
+  rjung: Note discussion at http://marc.info/?t=12518257461&r=1&w=2



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



DO NOT REPLY [Bug 47750] Loss of worker settings when changing via jkstatus

2009-09-01 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=47750



--- Comment #8 from robert.ma...@capita.co.uk 2009-09-01 13:47:08 PDT ---
(In reply to comment #6)
> Thanks for the config, will have a look.
> The request which triggered the second start was using the URL
> /EODPut/225/TEST.EOD225.20090826.txt
> and going to the same server name as all the other requests. Still reasoning,
> why this request started another (second) process.

This is a WebDAV location, and the particular call above writes a file into the
location - I did wonder if this was the trigger, but further experimentation
showed that:
1.  Calls to the WebDAV location don't usually trigger another start.
2.  Other URLs trigger the start.

Additionally WebDAV is using a different application pool, so in theory should
be separated from ISAPI redirect.  I also have another environment which
experiences the originally reported condition, and that doesn't have WebDAV on
it.

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- 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: svn commit: r810132 - /tomcat/tc6.0.x/trunk/STATUS.txt

2009-09-01 Thread sebb
On 01/09/2009, Larry Isaacs  wrote:
> Hi Filip,
>
>  I have a vague recollection about tripping over this issue while working on 
> the Tomcat support in the Eclipse Web Tools Platform.  WTP has its own 
> version(s) of VirtualWebappLoader, called WtpWebappLoader, to address it.  If 
> I'm recalling the issue correctly, pre-pending "file:/" worked on Windows, 
> but not on Linux since an absolute path ended up as "file://somedir/...".  I 
> believe with the change below, pre-pending "file://" will work for Linux, but 
> not for Windows, i.e. "file://C:/somedir/..." is going to have problems.  If 
> it helps, I went with the URL form "file:/somedir/..." and used the following 
> code snippet to get it working for Windows and Linux in WtpWebappLoader:
>
>
> String path = file.getAbsolutePath();
> if (path.startsWith("/")) {
> path = "file:" + path;
> } else {
> path = "file:/" + path;
> }
> if (file.isDirectory()) {
> addRepository(path + "/");

Also, perhaps consider using FileSystem.getPathSeparator() instead of "/" ?

> } else {
> addRepository(path);
> }
>
>  Cheers,
>
> Larry
>
>
>
>  > -Original Message-
>  > From: fha...@apache.org [mailto:fha...@apache.org]
>  > Sent: Tuesday, September 01, 2009 1:22 PM
>  > To: dev@tomcat.apache.org
>  > Subject: svn commit: r810132 - /tomcat/tc6.0.x/trunk/STATUS.txt
>  >
>  > Author: fhanik
>  > Date: Tue Sep  1 17:21:59 2009
>  > New Revision: 810132
>  >
>  > URL: http://svn.apache.org/viewvc?rev=810132&view=rev
>  > Log:
>  > proposal
>  >
>  > 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=810132
>  > &r1=810131&r2=810132&view=diff
>  > ===
>  > ===
>  > --- tomcat/tc6.0.x/trunk/STATUS.txt (original)
>  > +++ tomcat/tc6.0.x/trunk/STATUS.txt Tue Sep  1 17:21:59 2009
>  > @@ -321,3 +321,25 @@
>  >+1: funkman
>  >-1:
>  >
>  > +
>  > +* Fix VirtualWebappClassLoader file directives
>  > +  Index: java/org/apache/catalina/loader/VirtualWebappLoader.java
>  > +===
>  > +--- java/org/apache/catalina/loader/VirtualWebappLoader.java
>  >   (revision 810099)
>  >  java/org/apache/catalina/loader/VirtualWebappLoader.java
>  >   (working copy)
>  > +@@ -92,9 +92,9 @@
>  > + continue;
>  > + }
>  > + if (file.isDirectory()) {
>  > +-addRepository("file:/" + file.getAbsolutePath() +
>  > "/");
>  > ++addRepository("file://" + file.getAbsolutePath() +
>  > "/");
>  > + } else {
>  > +-addRepository("file:/" + file.getAbsolutePath());
>  > ++addRepository("file://" + file.getAbsolutePath());
>  > + }
>  > + }
>  > +
>  > +  +1: fhanik
>  > +  -1:
>  > +
>  > +
>  >
>  >
>  >
>  > -
>  > 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