2015-03-26 15:40 GMT+02:00 Ognjen Blagojevic <ognjen.d.blagoje...@gmail.com
>:
>
> On 26.3.2015 14:23, Ognjen Blagojevic wrote:
>>
>> On 26.3.2015 9:31, Violeta Georgieva wrote:
>>>
>>> The proposed 7.0.60 release is:
>>> [X] Broken - do not release
>>> [ ] Stable - go ahead and release as 7.0.60 Stable
>
> ...
>>
>> I'm still investigating why this doesn't work as expected.
>
>
> It was introduced in 1662994, in method
AbstractEndpoint.testServerCipherSuitesOrderSupport (reformatted for
clarity):
>
> if (JreCompat.isJre8Available()) {
>   throw new UnsupportedOperationException(
>     sm.getString("endpoint.jsse.cannotHonorServerCipherOrder"));
>
> Condition is wrong, it should throw exception only if Jre8 is NOT
available:
>
> if (!JreCompat.isJre8Available()) {
>   throw new UnsupportedOperationException(
>     sm.getString("endpoint.jsse.cannotHonorServerCipherOrder"));
>

I've just prepared a patch ;).

Index: AbstractEndpoint.java
===================================================================
--- AbstractEndpoint.java (revision 1669289)
+++ AbstractEndpoint.java (working copy)
@@ -654,7 +654,7 @@
     private void testServerCipherSuitesOrderSupport() {
         // Only test this feature if the user explicitly requested its use.
         if(!"".equals(getUseServerCipherSuitesOrder().trim())) {
-            if (JreCompat.isJre8Available()) {
+            if (!JreCompat.isJre8Available()) {
                 throw new UnsupportedOperationException(

 sm.getString("endpoint.jsse.cannotHonorServerCipherOrder"));
             }


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

Reply via email to