• Felix Lange's avatar
    p2p: enforce connection retry limit on server side (#19684) · c420dcb3
    Felix Lange authored
    The dialer limits itself to one attempt every 30s. Apply the same limit
    in Server and reject peers which try to connect too eagerly. The check
    against the limit happens right after accepting the connection.
    
    Further changes in this commit ensure we pass the Server logger
    down to Peer instances, discovery and dialState. Unit test logging now
    works in all Server tests.
    Unverified
    c420dcb3
util_test.go 1.56 KB
// Copyright 2019 The go-ethereum Authors
// This file is part of the go-ethereum library.
//
// The go-ethereum library is free software: you can redistribute it and/or modify
// it under the terms of the GNU Lesser General Public License as published by
// the Free Software Foundation, either version 3 of the License, or
// (at your option) any later version.
//
// The go-ethereum library is distributed in the hope that it will be useful,
// but WITHOUT ANY WARRANTY; without even the implied warranty of
// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
// GNU Lesser General Public License for more details.
//
// You should have received a copy of the GNU Lesser General Public License
// along with the go-ethereum library. If not, see <http://www.gnu.org/licenses/>.

package p2p

import (
	"testing"
	"time"
)

func TestExpHeap(t *testing.T) {
	var h expHeap

	var (
		basetime = time.Unix(4000, 0)
		exptimeA = basetime.Add(2 * time.Second)
		exptimeB = basetime.Add(3 * time.Second)
		exptimeC = basetime.Add(4 * time.Second)
	)
	h.add("a", exptimeA)
	h.add("b", exptimeB)
	h.add("c", exptimeC)

	if !h.nextExpiry().Equal(exptimeA) {
		t.Fatal("wrong nextExpiry")
	}
	if !h.contains("a") || !h.contains("b") || !h.contains("c") {
		t.Fatal("heap doesn't contain all live items")
	}

	h.expire(exptimeA.Add(1))
	if !h.nextExpiry().Equal(exptimeB) {
		t.Fatal("wrong nextExpiry")
	}
	if h.contains("a") {
		t.Fatal("heap contains a even though it has already expired")
	}
	if !h.contains("b") || !h.contains("c") {
		t.Fatal("heap doesn't contain all live items")
	}
}