Return-Path: X-Original-To: archive-asf-public-internal@cust-asf2.ponee.io Delivered-To: archive-asf-public-internal@cust-asf2.ponee.io Received: from cust-asf.ponee.io (cust-asf.ponee.io [163.172.22.183]) by cust-asf2.ponee.io (Postfix) with ESMTP id D0189200D3A for ; Tue, 31 Oct 2017 21:37:07 +0100 (CET) Received: by cust-asf.ponee.io (Postfix) id CEB851609E6; Tue, 31 Oct 2017 20:37:07 +0000 (UTC) Delivered-To: archive-asf-public@cust-asf.ponee.io Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by cust-asf.ponee.io (Postfix) with SMTP id 236BC1609EF for ; Tue, 31 Oct 2017 21:37:06 +0100 (CET) Received: (qmail 85814 invoked by uid 500); 31 Oct 2017 20:37:06 -0000 Mailing-List: contact issues-help@geode.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@geode.apache.org Delivered-To: mailing list issues@geode.apache.org Received: (qmail 85648 invoked by uid 99); 31 Oct 2017 20:37:05 -0000 Received: from pnap-us-west-generic-nat.apache.org (HELO spamd3-us-west.apache.org) (209.188.14.142) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 31 Oct 2017 20:37:05 +0000 Received: from localhost (localhost [127.0.0.1]) by spamd3-us-west.apache.org (ASF Mail Server at spamd3-us-west.apache.org) with ESMTP id 1D41B183172 for ; Tue, 31 Oct 2017 20:37:05 +0000 (UTC) X-Virus-Scanned: Debian amavisd-new at spamd3-us-west.apache.org X-Spam-Flag: NO X-Spam-Score: -99.202 X-Spam-Level: X-Spam-Status: No, score=-99.202 tagged_above=-999 required=6.31 tests=[KAM_ASCII_DIVIDERS=0.8, RP_MATCHES_RCVD=-0.001, SPF_PASS=-0.001, USER_IN_WHITELIST=-100] autolearn=disabled Received: from mx1-lw-us.apache.org ([10.40.0.8]) by localhost (spamd3-us-west.apache.org [10.40.0.10]) (amavisd-new, port 10024) with ESMTP id cOHT-vpp4URt for ; Tue, 31 Oct 2017 20:37:04 +0000 (UTC) Received: from mailrelay1-us-west.apache.org (mailrelay1-us-west.apache.org [209.188.14.139]) by mx1-lw-us.apache.org (ASF Mail Server at mx1-lw-us.apache.org) with ESMTP id 91C3B61188 for ; Tue, 31 Oct 2017 20:37:03 +0000 (UTC) Received: from jira-lw-us.apache.org (unknown [207.244.88.139]) by mailrelay1-us-west.apache.org (ASF Mail Server at mailrelay1-us-west.apache.org) with ESMTP id 37C8FE2650 for ; Tue, 31 Oct 2017 20:37:02 +0000 (UTC) Received: from jira-lw-us.apache.org (localhost [127.0.0.1]) by jira-lw-us.apache.org (ASF Mail Server at jira-lw-us.apache.org) with ESMTP id ECBBE2130A for ; Tue, 31 Oct 2017 20:37:00 +0000 (UTC) Date: Tue, 31 Oct 2017 20:37:00 +0000 (UTC) From: "ASF GitHub Bot (JIRA)" To: issues@geode.apache.org Message-ID: In-Reply-To: References: Subject: [jira] [Commented] (GEODE-3894) Create HandshakeRequest MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-JIRA-FingerPrint: 30527f35849b9dde25b450d4833f0394 archived-at: Tue, 31 Oct 2017 20:37:08 -0000 [ https://issues.apache.org/jira/browse/GEODE-3894?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16227489#comment-16227489 ] ASF GitHub Bot commented on GEODE-3894: --------------------------------------- galen-pivotal commented on a change in pull request #985: GEODE-3894: Create HandshakeRequest URL: https://github.com/apache/geode/pull/985#discussion_r148088332 ########## File path: geode-protobuf/src/main/java/org/apache/geode/internal/protocol/protobuf/ProtobufLocatorPipeline.java ########## @@ -23,28 +23,32 @@ import org.apache.geode.cache.IncompatibleVersionException; import org.apache.geode.distributed.internal.InternalLocator; import org.apache.geode.internal.cache.client.protocol.ClientProtocolProcessor; +import org.apache.geode.internal.protocol.state.ConnectionStateProcessor; import org.apache.geode.internal.protocol.MessageExecutionContext; +import org.apache.geode.internal.protocol.state.NoSecurityConnectionStateProcessor; import org.apache.geode.internal.protocol.statistics.ProtocolClientStatistics; @Experimental public final class ProtobufLocatorPipeline implements ClientProtocolProcessor { private final ProtocolClientStatistics statistics; private final InternalLocator locator; private final ProtobufStreamProcessor streamProcessor; + private final ConnectionStateProcessor locatorConnectionState; ProtobufLocatorPipeline(ProtobufStreamProcessor protobufStreamProcessor, ProtocolClientStatistics statistics, InternalLocator locator) { this.streamProcessor = protobufStreamProcessor; this.statistics = statistics; this.locator = locator; this.statistics.clientConnected(); + this.locatorConnectionState = new NoSecurityConnectionStateProcessor(); Review comment: I look forward to the day when we manage to implement security on the locator. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: users@infra.apache.org > Create HandshakeRequest > ----------------------- > > Key: GEODE-3894 > URL: https://issues.apache.org/jira/browse/GEODE-3894 > Project: Geode > Issue Type: Improvement > Components: client/server > Reporter: Brian Baynes > > Make Handshake a HandshakeRequest that is contained in a Message the same way other requests are handled. -- This message was sent by Atlassian JIRA (v6.4.14#64029)