Skip to content
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.

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

identify: add test for observed address handling #2828

Merged
merged 1 commit into from
Jun 9, 2024

Conversation

ivan4th
Copy link
Contributor

@ivan4th ivan4th commented Jun 8, 2024

This modifies TestObservedAddrManager to verify the fix in #2825

This modifies TestObservedAddrManager to verify the fix in libp2p#2825
Copy link
Member

@sukunrt sukunrt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @ivan4th

@sukunrt sukunrt merged commit 2e80f1e into libp2p:master Jun 9, 2024
11 checks passed
sukunrt pushed a commit that referenced this pull request Jun 12, 2024
This modifies TestObservedAddrManager to verify the fix in #2825
sukunrt pushed a commit that referenced this pull request Jun 12, 2024
This modifies TestObservedAddrManager to verify the fix in #2825
MarcoPolo added a commit that referenced this pull request Jun 12, 2024
* identify: Don't filter addr if remote is neither public nor private (#2820)

Updates the filterAddrs logic to no-op if the address is neither public nor private.

This fixes an issue in mocknet that assigns each node an address in the IPv6 discard prefix space. That doesn't interact well with this logic in identify.

The issue mocknet hits is that it filters out all received listen addresses and then doesn't remember any address for the peer.

* identify: fix bug in observed address handling (#2825)

* identify: add test for observed address handling (#2828)

This modifies TestObservedAddrManager to verify the fix in #2825

* libp2phttp: workaround for ResponseWriter's CloseNotifier (#2821)

* libp2phttp: workaround for CloseNotifier

* Add lintignore

* circuitv2: improve voucher validation (#2826)

* webrtc: fix ufrag prefix for dialing (#2832)

* webrtc: add a test for establishing many connections (#2801)

Update pion/ice to include the fix for out of order 
ConnectionState update callbacks

* release v0.35.1

---------

Co-authored-by: Marco Munizaga <[email protected]>
Co-authored-by: Ivan Shvedunov <[email protected]>
MarcoPolo pushed a commit that referenced this pull request Jun 13, 2024
This modifies TestObservedAddrManager to verify the fix in #2825
MarcoPolo pushed a commit that referenced this pull request Jun 13, 2024
This modifies TestObservedAddrManager to verify the fix in #2825
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants