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

require repair request signature #31888

Merged
merged 1 commit into from
Jun 12, 2023

Conversation

jbiseda
Copy link
Contributor

@jbiseda jbiseda commented May 31, 2023

Problem

Signed repair requests are not required on all cluster types. Versions prior to 1.13.6 do not send signed repair requests. Once the cluster validator floor version is at least 1.13.6, repair requests should be required to be signed.

Summary of Changes

Remove ClusterType gating for signed repair requests.

Fixes #

@jbiseda jbiseda requested a review from willhickey May 31, 2023 00:32
@jbiseda jbiseda added v1.14 v1.16 PRs that should be backported to v1.16 labels May 31, 2023
@codecov
Copy link

codecov bot commented May 31, 2023

Codecov Report

Merging #31888 (e18e238) into master (450e7c2) will decrease coverage by 0.1%.
The diff coverage is 22.2%.

@@            Coverage Diff            @@
##           master   #31888     +/-   ##
=========================================
- Coverage    81.9%    81.9%   -0.1%     
=========================================
  Files         764      764             
  Lines      208095   208085     -10     
=========================================
- Hits       170573   170464    -109     
- Misses      37522    37621     +99     

@jbiseda jbiseda removed the v1.14 label Jun 7, 2023
@jbiseda jbiseda force-pushed the repair-require-signed-requests branch from 4bde894 to e18e238 Compare June 10, 2023 00:36
@jbiseda jbiseda marked this pull request as ready for review June 10, 2023 02:55
@jbiseda jbiseda requested review from behzadnouri and steviez June 10, 2023 05:53
Copy link
Contributor

@willhickey willhickey left a comment

Choose a reason for hiding this comment

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

LGTM

@jbiseda jbiseda merged commit b06df6a into solana-labs:master Jun 12, 2023
mergify bot pushed a commit that referenced this pull request Jun 12, 2023
@jbiseda jbiseda deleted the repair-require-signed-requests branch June 12, 2023 23:56
jbiseda added a commit that referenced this pull request Jun 13, 2023
require repair request signature (#31888)

(cherry picked from commit b06df6a)

Co-authored-by: Jeff Biseda <jbiseda@gmail.com>
jeffwashington pushed a commit to HaoranYi/solana that referenced this pull request Jun 13, 2023
wen-coding added a commit to wen-coding/solana that referenced this pull request Aug 15, 2023
wen-coding pushed a commit to wen-coding/solana that referenced this pull request Aug 15, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
v1.16 PRs that should be backported to v1.16
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants