https://issues.apache.org/bugzilla/show_bug.cgi?id=56825
Mark Thomas changed:
What|Removed |Added
Status|REOPENED|RESOLVED
Resolution|---
https://issues.apache.org/bugzilla/show_bug.cgi?id=56825
--- Comment #10 from Mark Thomas ---
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 sure that the cu
https://issues.apache.org/bugzilla/show_bug.cgi?id=56825
--- Comment #9 from jlmonteiro ---
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 ActionCode.REQ_SSL_ATTRIBUTE.
https://issues.apache.org/bugzilla/show_bug.cgi?id=56825
Konstantin Kolinko changed:
What|Removed |Added
Status|RESOLVED|REOPENED
Resolution|F
https://issues.apache.org/bugzilla/show_bug.cgi?id=56825
Mark Thomas changed:
What|Removed |Added
Status|REOPENED|RESOLVED
Resolution|---
https://issues.apache.org/bugzilla/show_bug.cgi?id=56825
Konstantin Kolinko changed:
What|Removed |Added
Status|RESOLVED|REOPENED
Resolution|F
https://issues.apache.org/bugzilla/show_bug.cgi?id=56825
--- Comment #5 from jlmonteiro ---
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
Mark Thomas changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://issues.apache.org/bugzilla/show_bug.cgi?id=56825
--- Comment #3 from jlmonteiro ---
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:
You are the assignee for
https://issues.apache.org/bugzilla/show_bug.cgi?id=56825
--- Comment #2 from Mark Thomas ---
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 patch
harder to r
https://issues.apache.org/bugzilla/show_bug.cgi?id=56825
--- Comment #1 from jlmonteiro ---
Created attachment 31885
--> https://issues.apache.org/bugzilla/attachment.cgi?id=31885&action=edit
Patch with the test to reproduce and a fix
--
You are receiving this mail because:
You are the assign
https://issues.apache.org/bugzilla/show_bug.cgi?id=56825
jlmonteiro changed:
What|Removed |Added
OS||All
--
You are receiving this mail b
12 matches
Mail list logo
