Skip to content

Commit 02b87d9

Browse files
thinkAfCodGrapeBaBa
authored andcommitted
fix: test case and upgrade utp-go version
1 parent 210a81d commit 02b87d9

File tree

4 files changed

+18
-1
lines changed

4 files changed

+18
-1
lines changed

go.mod

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,7 @@ require (
5555
github.com/mattn/go-sqlite3 v1.14.18
5656
github.com/naoina/toml v0.1.2-0.20170918210437-9fafd6967416
5757
github.com/olekukonko/tablewriter v0.0.5
58-
github.com/optimism-java/utp-go v0.0.0-20240530085325-d8dd9d262631
58+
github.com/optimism-java/utp-go v0.0.0-20240531024756-00da67044c50
5959
github.com/peterh/liner v1.1.1-0.20190123174540-a2c9a5303de7
6060
github.com/protolambda/bls12-381-util v0.1.0
6161
github.com/protolambda/zrnt v0.32.2

go.sum

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -433,6 +433,10 @@ github.com/optimism-java/utp-go v0.0.0-20240309041853-b6b3a0dea581 h1:ZxgrtI0xIw
433433
github.com/optimism-java/utp-go v0.0.0-20240309041853-b6b3a0dea581/go.mod h1:DZ0jYzLzt4ZsCmhI/iqYgGFoNx45OfpEoKzXB8HVALQ=
434434
github.com/optimism-java/utp-go v0.0.0-20240530085325-d8dd9d262631 h1:01AecSuOSS6fsIU/oTVG/C70hIl3xPen99qy2hGr57w=
435435
github.com/optimism-java/utp-go v0.0.0-20240530085325-d8dd9d262631/go.mod h1:DZ0jYzLzt4ZsCmhI/iqYgGFoNx45OfpEoKzXB8HVALQ=
436+
github.com/optimism-java/utp-go v0.0.0-20240531021243-e12d25b6be38 h1:t0gRqfM7wUrFyryagUpw4TmYY0DLt+rjPaBd92i+W2M=
437+
github.com/optimism-java/utp-go v0.0.0-20240531021243-e12d25b6be38/go.mod h1:DZ0jYzLzt4ZsCmhI/iqYgGFoNx45OfpEoKzXB8HVALQ=
438+
github.com/optimism-java/utp-go v0.0.0-20240531024756-00da67044c50 h1:I1jGQkNEWq7BTFZkCJKLDrqFLC1jR3EC7jz3to4kpLg=
439+
github.com/optimism-java/utp-go v0.0.0-20240531024756-00da67044c50/go.mod h1:DZ0jYzLzt4ZsCmhI/iqYgGFoNx45OfpEoKzXB8HVALQ=
436440
github.com/optimism-java/zrnt v0.32.4-0.20240415084906-d9dbf06b32f7 h1:ZTQWXQ8xblCRUXhZs3h5qrBMSAHe8iNH7BG7a7IVFlI=
437441
github.com/optimism-java/zrnt v0.32.4-0.20240415084906-d9dbf06b32f7/go.mod h1:A0fezkp9Tt3GBLATSPIbuY4ywYESyAuc/FFmPKg8Lqs=
438442
github.com/peterh/liner v1.1.1-0.20190123174540-a2c9a5303de7 h1:oYW+YCJ1pachXTQmzR3rNLYGGz4g/UgFcjb28p/viDM=

p2p/discover/portal_protocol.go

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1044,6 +1044,7 @@ func (p *PortalProtocol) handleFindContent(id enode.ID, addr *net.UDPAddr, reque
10441044
connIdSend := connId.SendId()
10451045

10461046
go func(bctx context.Context) {
1047+
defer p.connIdGen.Remove(connId)
10471048
for {
10481049
select {
10491050
case <-bctx.Done():
@@ -1166,6 +1167,7 @@ func (p *PortalProtocol) handleOffer(id enode.ID, addr *net.UDPAddr, request *po
11661167
connIdSend := connId.SendId()
11671168

11681169
go func(bctx context.Context) {
1170+
defer p.connIdGen.Remove(connId)
11691171
for {
11701172
select {
11711173
case <-bctx.Done():

p2p/discover/portal_protocol_test.go

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,7 @@ import (
1010
"testing"
1111
"time"
1212

13+
"github.com/ethereum/go-ethereum/p2p/enr"
1314
"github.com/ethereum/go-ethereum/portalnetwork/storage"
1415
"github.com/prysmaticlabs/go-bitfield"
1516
"golang.org/x/exp/slices"
@@ -78,6 +79,7 @@ func setupLocalPortalNode(addr string, bootNodes []*enode.Node) (*PortalProtocol
7879
}
7980
}
8081
}
82+
localNode.Set(enr.UDP(addr1.Port))
8183

8284
discV5, err := ListenV5(conn, localNode, discCfg)
8385
if err != nil {
@@ -113,6 +115,15 @@ func TestPortalWireProtocolUdp(t *testing.T) {
113115
assert.NoError(t, err)
114116
time.Sleep(10 * time.Second)
115117

118+
node1.putCacheNodeId(node2.localNode.Node())
119+
node1.putCacheNodeId(node3.localNode.Node())
120+
121+
node2.putCacheNodeId(node1.localNode.Node())
122+
node2.putCacheNodeId(node3.localNode.Node())
123+
124+
node3.putCacheNodeId(node1.localNode.Node())
125+
node3.putCacheNodeId(node2.localNode.Node())
126+
116127
udpAddrStr1 := fmt.Sprintf("%s:%d", node1.localNode.Node().IP(), node1.localNode.Node().UDP())
117128
udpAddrStr2 := fmt.Sprintf("%s:%d", node2.localNode.Node().IP(), node2.localNode.Node().UDP())
118129

0 commit comments

Comments
 (0)