From a747232c88a3a7e42b5b40432d918d594b7068b6 Mon Sep 17 00:00:00 2001 From: Artem Barger Date: Thu, 15 Jun 2017 11:51:47 +0300 Subject: [PATCH] [FAB-4744] Remove unused code from gossip/comm There are few variables and unused values in gosisp communication module which could be cleaned up, this commit takes care to remove them. Change-Id: Ibedcfce97ac480afbdb465969ce99c1d3b60559a Signed-off-by: Artem Barger --- gossip/comm/comm_impl.go | 1 - gossip/comm/comm_test.go | 2 +- 2 files changed, 1 insertion(+), 2 deletions(-) diff --git a/gossip/comm/comm_impl.go b/gossip/comm/comm_impl.go index e331f3335b..31613b4b3a 100644 --- a/gossip/comm/comm_impl.go +++ b/gossip/comm/comm_impl.go @@ -155,7 +155,6 @@ type commImpl struct { subscriptions []chan proto.ReceivedMessage port int stopping int32 - skipHandshake bool } func (c *commImpl) createConnection(endpoint string, expectedPKIID common.PKIidType) (*connection, error) { diff --git a/gossip/comm/comm_test.go b/gossip/comm/comm_test.go index b37014fe31..7cb3f93e5b 100644 --- a/gossip/comm/comm_test.go +++ b/gossip/comm/comm_test.go @@ -112,7 +112,6 @@ func newCommInstance(port int, sec api.MessageCryptoService) (Comm, error) { } type msgMutator func(*proto.SignedGossipMessage) *proto.SignedGossipMessage -type msgConsumer func(*proto.SignedGossipMessage) type tlsType int @@ -221,6 +220,7 @@ func TestHandshake(t *testing.T) { // Positive path 1 - check authentication without TLS ll, err := net.Listen("tcp", fmt.Sprintf("%s:%d", "", 9611)) + assert.NoError(t, err) s := grpc.NewServer() go s.Serve(ll)