Skip to content

fix: prevent self dial #5306

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
gacevicljubisa merged 29 commits into from
Jan 2, 2026
Merged

fix: prevent self dial #5306

gacevicljubisa merged 29 commits into from
Jan 2, 2026

Conversation

@gacevicljubisa
Copy link
Member

@gacevicljubisa gacevicljubisa commented Dec 16, 2025

Checklist

  • I have read the coding guide.
  • My change requires a documentation update, and I have done it.
  • I have added tests to cover my changes.
  • I have filled out the description and linked the related issues.

Description

Open API Spec Version Changes (if applicable)

Motivation and Context (Optional)

Related Issue (Optional)

Screenshots (if appropriate):

janos and others added 27 commits December 4, 2025 17:07
@janos
hive: remove ping
e98adba
@janos
pullsync: remove make offer timeout
a8faeb3
@gacevicljubisa
fix(libp2p): consolidate AddrsFactory logic for all transports
f1a869a
@gacevicljubisa
fix(libp2p): increase ping timeout to 5s
2f10842
@gacevicljubisa
fix(libp2p): remove ping after handshake
c5fbf5a
@gacevicljubisa
fix(hive): remove unused context
a519468
@gacevicljubisa
fix: skip self connect
ada6913
@gacevicljubisa
fix(pushsync): return after FullClose in handler
dcf09a7
@gacevicljubisa
chore: revert go-storage-incentives-abi to v0.9.4
5d4ee00
@gacevicljubisa
fix(hive): preallocate peersToAdd slice
5a0524d
@gacevicljubisa
Merge branch 'ljubisa/fix-autotls' into ljubisa/fix-autotls-skip-self…
3f48247 
…connect
@gacevicljubisa
fix(pushsync): return after FullClose in handler
b27bd1b
@gacevicljubisa
fix(libp2p): improve flow for addrFactory when we have ws and wss
805a160
@gacevicljubisa
Merge branch 'ljubisa/fix-autotls' into ljubisa/fix-autotls-skip-self…
21538f9 
…connect
@gacevicljubisa
fix(libp2p): remove TODO comment
8598fd4
@gacevicljubisa
Merge branch 'ljubisa/fix-autotls' into ljubisa/fix-autotls-skip-self…
9520684 
…connect
@gacevicljubisa
Merge branch 'master' into autotls-v2
e92eb16
@gacevicljubisa
chore: bump deps
b3e9f7d
@gacevicljubisa
fix(kademlia): prevent self dial on AddPeers handler
8d050a5
@gacevicljubisa
fix(libp2p): remove t.Parallel() from tests
c43235b
@gacevicljubisa
fix: add unit tests for self connect
9543e61
@gacevicljubisa
fix(libp2p): call Disconnect if host not connected
96e799e
@gacevicljubisa
Merge branch 'autotls-v2' into ljubisa/fix-autotls-skip-selfconnect
d415e76
@gacevicljubisa
fix(libp2p): increase limit per ip connections
74b8299
@gacevicljubisa
fix(libp2p): use custom rate limiter
c2012b9
@gacevicljubisa
fix(libp2p): close only failing connection on handshake error
9a4660e
@gacevicljubisa
fix(libp2p): always enable websocket transport for dialing
a7892f6
@gacevicljubisa gacevicljubisa marked this pull request as ready for review December 29, 2025 09:05
@gacevicljubisa gacevicljubisa merged commit b756dbe into autotls-v2 Jan 2, 2026
13 of 14 checks passed
@gacevicljubisa gacevicljubisa deleted the ljubisa/fix-autotls-skip-selfconnect branch January 2, 2026 09:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@janos janos janos approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@gacevicljubisa @janos