RE: 5.5.4 ?

2004-10-26 Thread Shapira, Yoav

Hi,
Looks like a RESOLVED-WONTFIX ;)

Yoav Shapira http://www.yoavshapira.com


-Original Message-
From: Ben Souther [mailto:[EMAIL PROTECTED]
Sent: Monday, October 25, 2004 5:06 PM
To: Tomcat Developers List
Subject: Re: 5.5.4 ?

Yes, I had tested it a little earlier, and it doesn't work. The path
 would apparently have to be encoded in the same way as the URL.
OK, let me know if I can help.


 Quite
 frankly, I'm not sure we're going to do this, since the encoding on
the
 client side is quite unpredictable.
Other than helping out and trying to learn the code, I have no interest
in seeing this one resolved.

IMHO: it's a little absurd to have to support spaces in a context path
since a browser will never send a URL with a space in it.
-Ben






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




This e-mail, including any attachments, is a confidential business communication, and 
may contain information that is confidential, proprietary and/or privileged.  This 
e-mail is intended only for the individual(s) to whom it is addressed, and may not be 
saved, copied, printed, disclosed or used by anyone else.  If you are not the(an) 
intended recipient, please immediately delete this e-mail from your computer system 
and notify the sender.  Thank you.


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



RE: 5.5.4 ?

2004-10-26 Thread Ben Souther
One way to make things more consistent without affecting performance
would be for the deployer not to deploy apps with spaces in the context
path.  This would be a lot easier to debug than the current behavior and
may prevent this bug from being reported again.



On Tue, 2004-10-26 at 08:45, Shapira, Yoav wrote:
 Hi,
 Looks like a RESOLVED-WONTFIX ;)
 
 Yoav Shapira http://www.yoavshapira.com
  
 
 -Original Message-
 From: Ben Souther [mailto:[EMAIL PROTECTED]
 Sent: Monday, October 25, 2004 5:06 PM
 To: Tomcat Developers List
 Subject: Re: 5.5.4 ?
 
 Yes, I had tested it a little earlier, and it doesn't work. The path
  would apparently have to be encoded in the same way as the URL.
 OK, let me know if I can help.
 
 
  Quite
  frankly, I'm not sure we're going to do this, since the encoding on
 the
  client side is quite unpredictable.
 Other than helping out and trying to learn the code, I have no interest
 in seeing this one resolved.
 
 IMHO: it's a little absurd to have to support spaces in a context path
 since a browser will never send a URL with a space in it.
 -Ben
 
 
 
 
 
 
 -
 To unsubscribe, e-mail: [EMAIL PROTECTED]
 For additional commands, e-mail: [EMAIL PROTECTED]
 
 
 
 
 This e-mail, including any attachments, is a confidential business communication, 
 and may contain information that is confidential, proprietary and/or privileged.  
 This e-mail is intended only for the individual(s) to whom it is addressed, and may 
 not be saved, copied, printed, disclosed or used by anyone else.  If you are not 
 the(an) intended recipient, please immediately delete this e-mail from your computer 
 system and notify the sender.  Thank you.
 
 
 -
 To unsubscribe, e-mail: [EMAIL PROTECTED]
 For additional commands, e-mail: [EMAIL PROTECTED]
 
 


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



RE: 5.5.4 ?

2004-10-25 Thread Shapira, Yoav

Hi,

What are the plans for 5.5.4 ?

I want to resolve (either fix or invalid, whatever) Bugzilla 31090
(space in context name makes session IDs crap, 31372
(AuthenticatorBase#register method), the couple of doc items, and
possibly 31656 (make Tomcat build with Struts 1.2).  This week looks
lighter at work so next weekend seems like something good to shoot for.

How about Saturday, October 30th (cut time TBD) for the 5.5.4 release?

Yoav



This e-mail, including any attachments, is a confidential business communication, and 
may contain information that is confidential, proprietary and/or privileged.  This 
e-mail is intended only for the individual(s) to whom it is addressed, and may not be 
saved, copied, printed, disclosed or used by anyone else.  If you are not the(an) 
intended recipient, please immediately delete this e-mail from your computer system 
and notify the sender.  Thank you.


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



Re: 5.5.4 ?

2004-10-25 Thread Remy Maucherat
Shapira, Yoav wrote:
Hi,
 

What are the plans for 5.5.4 ?
   

I want to resolve (either fix or invalid, whatever) Bugzilla 31090
 

I should have fixed that one, but I'm not sure, so someone needs to test it.
(space in context name makes session IDs crap, 31372
(AuthenticatorBase#register method), the couple of doc items, and
 

So that there are no surprises, I'm -1 for the patch proposed in the bug 
report.

possibly 31656 (make Tomcat build with Struts 1.2).  This week looks
lighter at work so next weekend seems like something good to shoot for.
 

Is it actually better ? ;) (= faster startup, for example)
How about Saturday, October 30th (cut time TBD) for the 5.5.4 release?
 

Ok.
Rémy
-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


RE: 5.5.4 ?

2004-10-25 Thread Shapira, Yoav

Hi,

(space in context name makes session IDs crap, 31372
(AuthenticatorBase#register method), the couple of doc items, and


So that there are no surprises, I'm -1 for the patch proposed in the
bug
report.

I saw your comments on this issue for another bug, and was hoping you
could post them (even if it's just copy/paste) to 31372 and then any one
of us could resolve it as invalid.

possibly 31656 (make Tomcat build with Struts 1.2).  This week looks
lighter at work so next weekend seems like something good to shoot
for.


Is it actually better ? ;) (= faster startup, for example)

I don't know ;)  The change is only to make sure it compiles with 1.2
(without breaking 1.1 compilation).  It's not necessarily to ship with
1.2.  I planned to test compilation only.  It's a forward-looking change
to when we need/want a Struts 1.2 feature ;)

Yoav



This e-mail, including any attachments, is a confidential business communication, and 
may contain information that is confidential, proprietary and/or privileged.  This 
e-mail is intended only for the individual(s) to whom it is addressed, and may not be 
saved, copied, printed, disclosed or used by anyone else.  If you are not the(an) 
intended recipient, please immediately delete this e-mail from your computer system 
and notify the sender.  Thank you.


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



Re: 5.5.4 ?

2004-10-25 Thread Ben Souther
On Mon, 2004-10-25 at 11:48, Remy Maucherat wrote:
 Shapira, Yoav wrote:
 
 Hi,
 
   
 
 What are the plans for 5.5.4 ?
 
 
 
 I want to resolve (either fix or invalid, whatever) Bugzilla 31090
   
 
 I should have fixed that one, but I'm not sure, so someone needs to test it.
Just tested with:
jakarta-tomcat-5-bin-20041024.tar.gz
and the problem still exists.
I can test build from the CVS head tonight and test if you think it's
different.

-Ben









 (space in context name makes session IDs crap, 31372
 (AuthenticatorBase#register method), the couple of doc items, and
   
 
 So that there are no surprises, I'm -1 for the patch proposed in the bug 
 report.
 
 possibly 31656 (make Tomcat build with Struts 1.2).  This week looks
 lighter at work so next weekend seems like something good to shoot for.
   
 
 Is it actually better ? ;) (= faster startup, for example)
 
 How about Saturday, October 30th (cut time TBD) for the 5.5.4 release?
   
 
 Ok.
 
 Rmy
 
 
 -
 To unsubscribe, e-mail: [EMAIL PROTECTED]
 For additional commands, e-mail: [EMAIL PROTECTED]
 
 


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



Re: 5.5.4 ?

2004-10-25 Thread Remy Maucherat
Ben Souther wrote:
Just tested with:
jakarta-tomcat-5-bin-20041024.tar.gz
and the problem still exists.
I can test build from the CVS head tonight and test if you think it's
different.
Yes, I had tested it a little earlier, and it doesn't work. The path 
would apparently have to be encoded in the same way as the URL. Quite 
frankly, I'm not sure we're going to do this, since the encoding on the 
client side is quite unpredictable.

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


Re: 5.5.4 ?

2004-10-25 Thread Ben Souther
Yes, I had tested it a little earlier, and it doesn't work. The path 
 would apparently have to be encoded in the same way as the URL.
OK, let me know if I can help.


 Quite 
 frankly, I'm not sure we're going to do this, since the encoding on the 
 client side is quite unpredictable.
Other than helping out and trying to learn the code, I have no interest
in seeing this one resolved.

IMHO: it's a little absurd to have to support spaces in a context path
since a browser will never send a URL with a space in it.
-Ben






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