Bruce Schuchardt commented on GEODE-4450:

Undoing Jinmei's change to CacheClientNotifier.registerGFEClient() fixes this 


I modified the test ClientRegisterInterestAuthDUnitTest.testRegisterInterest to 
add a security-client-dhalgo setting in the client and server to reproduce the 

> setting a client/server diffie-hellman algorithm breaks client/server 
> subscriptions
> -----------------------------------------------------------------------------------
>                 Key: GEODE-4450
>                 URL: https://issues.apache.org/jira/browse/GEODE-4450
>             Project: Geode
>          Issue Type: Improvement
>          Components: client/server
>            Reporter: Bruce Schuchardt
>            Priority: Major
> Having found that there are no tests for the security-client-dhalgo setting I 
> modified a test to use it.  The client/server handshake in the subscription 
> thread (cache client updater) hung on the server side trying to read client 
> credentials.
> I tracked this down to CacheClientNotifier.registerClient, which sends a 105 
> byte to the client along with some other bytes in its writeMessage() method.  
> The client isn't expecting this message and interprets the 105 as a failure 
> to register with the server.  The client then abandons the handshake and the 
> server hangs until its read times out.
> See also GEODE-4353, which wants to deprecate this broken feature.

This message was sent by Atlassian JIRA

Reply via email to