DO NOT REPLY [Bug 40177] - RequestDumperValve causes getCharacterEncoding to be called

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

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution||FIXED




--- Additional Comments From [EMAIL PROTECTED]  2006-12-26 06:40 ---
I see that the Valve configuration reference already has a strong note to this
effect.  I added the same note to the Valve's JavaDoc header, and to server.xml.
 I've also marked the RequestDumperValve class as deprecated and told people to
use the RequestDumperFilter instead.

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

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

DO NOT REPLY [Bug 40177] - RequestDumperValve causes getCharacterEncoding to be called

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

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|RESOLVED|REOPENED
 Resolution|FIXED   |




--- Additional Comments From [EMAIL PROTECTED]  2006-12-26 18:59 ---
Hi Yoav.

Making the Javadocs more informative is indeed helpful.  But, both Valves
and Filters exist separately because they are useful for different use cases:
Valves are useful when the administrator wants to do something with requests
at the servlet container level, without modifying webapps that may or may not
be owned or written by the administrator.  Filters are useful when you are the
webapp author, or when you're able and willing to modify the webapp, to do
something with requests.  There are valid uses for each, and the administrator
should choose for themselves from the available options.  I have found
RequestDumperValve useful on a number of occasions where I wanted more info
about the requests/responses to web applications I've developed, plus
webapps that I didn't write.  That includes webapps that were localized and
internationalized (in development, not every bug needs to be investigated
in character sets other than ISO8859-1).  The RequestDumperValve has been
around for quite a while now, and I find it quite useful, even if it doesn't
do everything.

Reopening to request the undeprecation of RequestDumperValve.


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

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

DO NOT REPLY [Bug 40177] - RequestDumperValve causes getCharacterEncoding to be called

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

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|REOPENED|RESOLVED
 Resolution||FIXED




--- Additional Comments From [EMAIL PROTECTED]  2006-12-27 04:26 ---
I've reluctantly un-deprecated and amended the change log as you request.

However, I mostly disagree with your statements: I think Valves are largely
out-dated and should be phased out in favor or portable solutions where
possible.  They don't exist separately because they're useful: they exist
separately because of the historical timeline in which they were developed.

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

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

DO NOT REPLY [Bug 40177] - RequestDumperValve causes getCharacterEncoding to be called

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

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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-27 11:43 ---
Thanks Yoav.

> I think Valves are largely out-dated and should be phased out in favor
> or portable solutions where possible.

I agree with that, but even if 80% or 90% of Valves are better off as
Filters, some still aren't.  As a developer I have found many cases over the
years where I needed to write code that could not be part of a webapp, or that
had to use Tomcat's code to do something.  Sometimes (rarely, but sometimes)
the code has to run as part of Tomcat, and Valves are better documented at
this point than s are.  Since it's not a servlet, nor a Filter, we
know that it's not portable to other servlet containers.  But, if your company
has already decided to use Tomcat, this is okay.  Not much code needs to go
into a Valve, usually.  And while debugging a webapp that is already packaged,
and/or already deployed, being able to just add a line or two to server.xml
to enable, say, RequestDumperValve (which already exists in Tomcat, ready to
run) helps quite a bit because it is a quick way to investigate a bug.  This
allows us to skip compiling a Filter, installing the class file somewhere,
plus we'd have to add both a  block and a  block to
the webapp's web.xml for it.  This makes the Valve more convenient in some
situations.  I wish Tomcat came with more useful Valve tools like this one,
not less.

I realize that Valves existed first, and that Filters followed.  Tomcat could
completely drop support for Valves and stay servlet compliant, however it
would take away a useful, helpful feature of Tomcat that people do use at
times.

Thanks again.  :)


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

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

DO NOT REPLY [Bug 40177] - RequestDumperValve causes getCharacterEncoding to be called

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

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Severity|normal  |enhancement




--- Additional Comments From [EMAIL PROTECTED]  2006-08-03 10:57 ---
This is a known, documented 'feature' of the RequestDumperValve.

If you have a patch for consideration, that would be great.

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

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

DO NOT REPLY [Bug 40177] - RequestDumperValve causes getCharacterEncoding to be called

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

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





--- Additional Comments From [EMAIL PROTECTED]  2006-08-03 14:54 ---
The only workaround I am aware of for this issue is to wait until the *end* of
the request and *then* call getParameter*() APIs and output request parameter
information.  Doing so at any point prior to this can mess up the application by
prematurely consuming the input stream and/or locking the request character
encoding.

I wrote a servlet request filter to do such things, provide JMX access, run
across all servlet engines, etc -- and thus don't use RequestDumperValve any
more.  Further I suggest to all developers I work with to avoid it except for
light troubleshooting as it causes more troubles to shoot :-)

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

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