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

Fix Peer Sharing issue #4642 #4644

Merged
merged 4 commits into from
Oct 31, 2023
Merged

Conversation

bolt12
Copy link
Contributor

@bolt12 bolt12 commented Aug 17, 2023

Fixes #4642 by simplifying the PeerSharing options, i.e. removes PeerSharingPrivate making the handshake negotiation symmetric.

I also updated the wiki page to keep track of these changes on the design docs: https://github.com/input-output-hk/ouroboros-network/wiki/Peer-Sharing-Implementation-Plan#post-implementation-notes

@bolt12 bolt12 requested a review from coot as a code owner August 17, 2023 08:50
@bolt12 bolt12 added networking peer-sharing Issues / PRs related to peer sharing labels Aug 17, 2023
@bolt12 bolt12 force-pushed the bolt12/fix-peerSharing-handshake branch from f9e9ab7 to 1c81439 Compare August 17, 2023 10:15
@bolt12 bolt12 force-pushed the bolt12/fix-peerSharing-handshake branch 2 times, most recently from 8ed5dbc to 12d0e25 Compare September 4, 2023 13:13
@bolt12 bolt12 force-pushed the bolt12/fix-peerSharing-handshake branch 2 times, most recently from c7e9e85 to 7003b2a Compare September 18, 2023 14:19
@bolt12 bolt12 force-pushed the bolt12/fix-peerSharing-handshake branch from 7003b2a to da8df7f Compare September 18, 2023 15:00
@coot
Copy link
Contributor

coot commented Sep 20, 2023

Is this PR ready for review?

@bolt12
Copy link
Contributor Author

bolt12 commented Sep 20, 2023

Yes!

@bolt12 bolt12 force-pushed the bolt12/fix-peerSharing-handshake branch from 6d89f63 to 853fa9b Compare September 25, 2023 13:19
@bolt12 bolt12 linked an issue Sep 25, 2023 that may be closed by this pull request
@bolt12 bolt12 force-pushed the bolt12/fix-peerSharing-handshake branch 2 times, most recently from 1878384 to 66cbbb9 Compare September 26, 2023 11:51
ouroboros-network/CHANGELOG.md Outdated Show resolved Hide resolved
ouroboros-network/CHANGELOG.md Outdated Show resolved Hide resolved
@bolt12 bolt12 force-pushed the bolt12/fix-peerSharing-handshake branch from 4127255 to eb3d770 Compare October 6, 2023 09:38
@bolt12 bolt12 requested a review from coot October 6, 2023 09:41
@bolt12 bolt12 force-pushed the bolt12/fix-peerSharing-handshake branch 3 times, most recently from 4666ef6 to 7e73302 Compare October 11, 2023 10:30
@bolt12 bolt12 requested a review from coot October 12, 2023 15:13
@bolt12 bolt12 force-pushed the bolt12/fix-peerSharing-handshake branch from 7e73302 to 58f261f Compare October 12, 2023 15:30
@bolt12 bolt12 enabled auto-merge October 13, 2023 14:55
@bolt12 bolt12 force-pushed the bolt12/fix-peerSharing-handshake branch from 58f261f to 3b2eb01 Compare October 23, 2023 09:18
@bolt12 bolt12 force-pushed the bolt12/fix-peerSharing-handshake branch from 3b2eb01 to bdd7d06 Compare October 27, 2023 11:07
@bolt12 bolt12 force-pushed the bolt12/fix-peerSharing-handshake branch 3 times, most recently from e3b9ea9 to c2c4db8 Compare October 27, 2023 15:34
- Remove `PeerSharingPrivate` option and renamed others to
  `PeerSharingDisabled` and `PeerSharingEnabled`
- Add test to check that the bug is fixed
- Add new `NodeToNodeV_13`
- Renamed spec files and added V13 CDDL handshake and
  `NodeToNodeVersionData` specs
@bolt12 bolt12 force-pushed the bolt12/fix-peerSharing-handshake branch from c2c4db8 to 50bab68 Compare October 30, 2023 11:17
Copy link
Contributor

@coot coot left a comment

Choose a reason for hiding this comment

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

LGTM, just a few minor remarks.

ouroboros-network-protocols/CHANGELOG.md Outdated Show resolved Hide resolved
ouroboros-network/CHANGELOG.md Outdated Show resolved Hide resolved
@bolt12 bolt12 added this pull request to the merge queue Oct 31, 2023
@coot coot removed this pull request from the merge queue due to a manual request Oct 31, 2023
@bolt12 bolt12 enabled auto-merge October 31, 2023 15:31
@bolt12 bolt12 added this pull request to the merge queue Oct 31, 2023
Merged via the queue into master with commit 6296e40 Oct 31, 2023
8 of 9 checks passed
@bolt12 bolt12 deleted the bolt12/fix-peerSharing-handshake branch October 31, 2023 18:02
github-merge-queue bot pushed a commit to IntersectMBO/ouroboros-consensus that referenced this pull request Nov 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
peer-sharing Issues / PRs related to peer sharing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

The encoding of IPv6 addresses for Peersharing is wrong. PeerSharing negotiated even when it is disabled.
3 participants