You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@geode.apache.org by "ASF subversion and git services (JIRA)" <ji...@apache.org> on 2017/09/18 21:07:00 UTC

[jira] [Commented] (GEODE-3636) GatewaySender stop can hang with SSL enabled

    [ https://issues.apache.org/jira/browse/GEODE-3636?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16170720#comment-16170720 ] 

ASF subversion and git services commented on GEODE-3636:
--------------------------------------------------------

Commit 152994f5445fc2fb4ae8b38438af236fc65238f5 in geode's branch refs/heads/develop from [~lhughesgodfrey]
[ https://gitbox.apache.org/repos/asf?p=geode.git;h=152994f ]

GEODE-3636: Fixing hang shutting down gateway sender with SSL

The gateway sender shutdown code calls connection.destroy to interrupt the ack
reader thread. Connection.destroy was calling socket.shutDownOutput.
SSLSockets do not support socket.shutDownOutput() resulting in connections not
being destroyed. This left the AckReaderThread stuck on read holding a
readLock, which prevented the stopper thread from getting a writeLock.


> GatewaySender stop can hang with SSL enabled
> --------------------------------------------
>
>                 Key: GEODE-3636
>                 URL: https://issues.apache.org/jira/browse/GEODE-3636
>             Project: Geode
>          Issue Type: Bug
>          Components: wan
>    Affects Versions: 1.1.0, 1.1.1, 1.2.0
>            Reporter: Shelley Lynn Hughes-Godfrey
>
> SSLSockets throw the following exception if shutdownOutput() invoked:
> java.lang.UnsupportedOperationException: 
> The method shutdownOutput() is not supported in SSLSocket
> GatewaySender code tries to interrupt the AckReaderThread in the middle of a socket read by calling Connection.destroy. Unfortunately, that throws and then swallows this UnsupportedOperationException and doesn't close the socket when SSL is enabled.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)