https://issues.apache.org/bugzilla/show_bug.cgi?id=56825
--- Comment #9 from jlmonteiro jlmonte...@tomitribe.com ---
Hi,
(In reply to Konstantin Kolinko from comment #8)
Re-reviewing the changes in Tomcat 7 (revisions r1617447 r1620827 and
r1622328 ) I have a question.
There exists
https://issues.apache.org/bugzilla/show_bug.cgi?id=56825
--- Comment #10 from Mark Thomas ma...@apache.org ---
I've dug a little more into this. The unit test setting clientAuth=want is
hiding the fact that this still isn't quite right.
Some new API is going to be necessary. That said, I'm not
https://issues.apache.org/bugzilla/show_bug.cgi?id=56825
Mark Thomas ma...@apache.org changed:
What|Removed |Added
Status|REOPENED|RESOLVED
https://issues.apache.org/bugzilla/show_bug.cgi?id=56825
Mark Thomas ma...@apache.org changed:
What|Removed |Added
Status|REOPENED|RESOLVED
https://issues.apache.org/bugzilla/show_bug.cgi?id=56825
Konstantin Kolinko knst.koli...@gmail.com changed:
What|Removed |Added
Status|RESOLVED
https://issues.apache.org/bugzilla/show_bug.cgi?id=56825
Konstantin Kolinko knst.koli...@gmail.com changed:
What|Removed |Added
Status|RESOLVED
https://issues.apache.org/bugzilla/show_bug.cgi?id=56825
Mark Thomas ma...@apache.org changed:
What|Removed |Added
Status|NEW |RESOLVED
https://issues.apache.org/bugzilla/show_bug.cgi?id=56825
--- Comment #5 from jlmonteiro jlmonte...@tomitribe.com ---
Thank you very much Mark.
I'll check your commit so that I can avoid such mistakes next time.
--
You are receiving this mail because:
You are the assignee for the bug.
https://issues.apache.org/bugzilla/show_bug.cgi?id=56825
--- Comment #2 from Mark Thomas ma...@apache.org ---
Generally we apply patches to the latest release and then back-port. Therefore
patches should be against 8.0.x.
The current patch changes import order and white space which a) makes the
https://issues.apache.org/bugzilla/show_bug.cgi?id=56825
--- Comment #3 from jlmonteiro jlmonte...@tomitribe.com ---
Thanks for reviewing.
I'll checkout the 8.0.x trunk and submit a new patch.
I'll take care as well of the formatting (including imports).
--
You are receiving this mail because:
https://issues.apache.org/bugzilla/show_bug.cgi?id=56825
jlmonteiro jlmonte...@tomitribe.com changed:
What|Removed |Added
OS||All
--
You
https://issues.apache.org/bugzilla/show_bug.cgi?id=56825
--- Comment #1 from jlmonteiro jlmonte...@tomitribe.com ---
Created attachment 31885
-- https://issues.apache.org/bugzilla/attachment.cgi?id=31885action=edit
Patch with the test to reproduce and a fix
--
You are receiving this mail
12 matches
Mail list logo