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

Add nodeaddress and capabilities at getblocks request #2732

Conversation

ManfredKarrer
Copy link
Contributor

@ManfredKarrer ManfredKarrer commented Apr 17, 2019

When we request at startup the blocks from a seed which has not yet
received out capabilities or node address we will not get a response.
We do not want to depend on the state in the getData domain in the
p2p network by using only a seed which has already responded but we
prefer to make sure the seednode will get all the data by our request
to be able to respond.

…edCapabilitiesMessage

When we request at startup the blocks from a seed which has not yet
received out capabilities or node address we will not get a response.
We do not want to depend on the state in the getData domain in the
p2p network by using only a seed which has already responded but we
prefer to make sure the seednode will get all the data by our request
to be able to respond.
@ManfredKarrer ManfredKarrer requested review from sqrrm and ripcurlx April 17, 2019 03:53
@ManfredKarrer ManfredKarrer added this to the v1.1.0 milestone Apr 17, 2019
@ManfredKarrer ManfredKarrer changed the title Add nodeaddress at getblocks request Add nodeaddress and capabilities at getblocks request Apr 17, 2019
Copy link
Member

@sqrrm sqrrm left a comment

Choose a reason for hiding this comment

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

utACK

@ManfredKarrer ManfredKarrer merged commit bebb481 into bisq-network:master Apr 17, 2019
@ManfredKarrer ManfredKarrer deleted the add-nodeaddress-at-getblocks-request branch April 17, 2019 23:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants