Skip to content
This repository was archived by the owner on Jan 22, 2025. It is now read-only.

enforces hash domain for ping-pong protocol#28433

Merged
behzadnouri merged 1 commit intosolana-labs:masterfrom
behzadnouri:rm-extra-ping
Oct 18, 2022
Merged

enforces hash domain for ping-pong protocol#28433
behzadnouri merged 1 commit intosolana-labs:masterfrom
behzadnouri:rm-extra-ping

Conversation

@behzadnouri
Copy link
Copy Markdown
Contributor

@behzadnouri behzadnouri commented Oct 17, 2022

Problem

#27193 added hash domain to ping-pong protocol.
For backward compatibility responses both with and without domain were generated and accepted.
Now that all clusters are upgraded, this commit enforces the hash domain by removing the response without the domain.

Summary of Changes

enforce hash domain for ping-pong protocol

solana-labs#27193
added hash domain to ping-pong protocol.
For backward compatibility responses both with and without domain were
generated and accepted.
Now that all clusters are upgraded, this commit enforces the hash domain
by removing the response without the domain.
Copy link
Copy Markdown
Contributor

@jbiseda jbiseda left a comment

Choose a reason for hiding this comment

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

LGTM

@behzadnouri behzadnouri merged commit e283461 into solana-labs:master Oct 18, 2022
@behzadnouri behzadnouri deleted the rm-extra-ping branch October 18, 2022 18:17
mergify Bot pushed a commit that referenced this pull request Oct 18, 2022
#27193
added hash domain to ping-pong protocol.
For backward compatibility responses both with and without domain were
generated and accepted.
Now that all clusters are upgraded, this commit enforces the hash domain
by removing the response without the domain.

(cherry picked from commit e283461)
mergify Bot added a commit that referenced this pull request Oct 18, 2022
enforces hash domain for ping-pong protocol (#28433)

#27193
added hash domain to ping-pong protocol.
For backward compatibility responses both with and without domain were
generated and accepted.
Now that all clusters are upgraded, this commit enforces the hash domain
by removing the response without the domain.

(cherry picked from commit e283461)

Co-authored-by: behzad nouri <behzadnouri@gmail.com>
mergify Bot pushed a commit that referenced this pull request Nov 28, 2022
#27193
added hash domain to ping-pong protocol.
For backward compatibility responses both with and without domain were
generated and accepted.
Now that all clusters are upgraded, this commit enforces the hash domain
by removing the response without the domain.

(cherry picked from commit e283461)

# Conflicts:
#	gossip/src/cluster_info.rs
mergify Bot added a commit that referenced this pull request Nov 28, 2022
* enforces hash domain for ping-pong protocol (#28433)

#27193
added hash domain to ping-pong protocol.
For backward compatibility responses both with and without domain were
generated and accepted.
Now that all clusters are upgraded, this commit enforces the hash domain
by removing the response without the domain.

(cherry picked from commit e283461)

# Conflicts:
#	gossip/src/cluster_info.rs

* removes mergify merge conflicts

Co-authored-by: behzad nouri <behzadnouri@gmail.com>
gnapoli23 pushed a commit to gnapoli23/solana that referenced this pull request Dec 16, 2022
solana-labs#27193
added hash domain to ping-pong protocol.
For backward compatibility responses both with and without domain were
generated and accepted.
Now that all clusters are upgraded, this commit enforces the hash domain
by removing the response without the domain.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants