Skip to content

Use correct protocol when serializing messages in reply to getdata#2157

Merged
UdjinM6 merged 1 commit into
dashpay:developfrom
UdjinM6:fixgovgetdatareply
Jun 30, 2018
Merged

Use correct protocol when serializing messages in reply to getdata#2157
UdjinM6 merged 1 commit into
dashpay:developfrom
UdjinM6:fixgovgetdatareply

Conversation

@UdjinM6

@UdjinM6 UdjinM6 commented Jun 30, 2018

Copy link
Copy Markdown

Messages should be serialized according the protocol of the peer who asked us or otherwise peers running on other protocols won't be able to deserialize the message correctly.

Messages should be serialized according the protocol of the peer who asked us or otherwise peers running on other protocols won't be able to deserialize the message correctly.
@UdjinM6 UdjinM6 added the bug label Jun 30, 2018
@UdjinM6 UdjinM6 added this to the 12.3 milestone Jun 30, 2018

@nmarley nmarley left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

utACK

@gladcow

gladcow commented Jun 30, 2018

Copy link
Copy Markdown

utACK

@codablock codablock left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

utACK

@UdjinM6
UdjinM6 merged commit fda74b4 into dashpay:develop Jun 30, 2018
andvgal pushed a commit to energicryptocurrency/gen2-energi that referenced this pull request Jan 6, 2019
…ashpay#2157)

Messages should be serialized according the protocol of the peer who asked us or otherwise peers running on other protocols won't be able to deserialize the message correctly.
CryptoCentric pushed a commit to absolute-community/absolute that referenced this pull request Mar 8, 2019
…ashpay#2157)

Messages should be serialized according the protocol of the peer who asked us or otherwise peers running on other protocols won't be able to deserialize the message correctly.
@UdjinM6
UdjinM6 deleted the fixgovgetdatareply branch November 26, 2020 11:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants