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
sovereign extra header verifier in runtype components #6140
sovereign extra header verifier in runtype components #6140
Conversation
…factor-sov-runtype-comp-16 # Conflicts: # factory/runType/sovereignRunTypeComponents.go # testscommon/components/components.go
…factor-sov-runtype-comp-16 # Conflicts: # testscommon/components/components.go
@@ -202,7 +196,11 @@ func (rcf *sovereignRunTypeComponentsFactory) Create() (*runTypeComponents, erro | |||
|
|||
expiryTime := time.Second * time.Duration(rcf.sovConfig.OutgoingSubscribedEvents.TimeToWaitForUnconfirmedOutGoingOperationInSeconds) | |||
|
|||
err = rtc.extraHeaderSigVerifierHandler.RegisterExtraHeaderSigVerifier(rcf.extraVerifier) | |||
sovHeaderSigVerifier, err := headerCheck.NewSovereignHeaderSigVerifier(rcf.cryptoComponents.BlockSigner()) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this also removed from normal run type ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Normal RunType doesn't use an extra header sig verifier, it's just:
extraHeaderSigVerifierHandler: headerCheck.NewExtraHeaderSigVerifierHolder(),
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The name of the variable was wrong and confused me;
extraHeaderSigVerifierHandler -> this is a holder, not a handler, please rename it
…factor-sov-runtype-comp-16
…factor-sov-runtype-comp-16
…factor-sov-runtype-comp-16
@@ -202,7 +196,11 @@ func (rcf *sovereignRunTypeComponentsFactory) Create() (*runTypeComponents, erro | |||
|
|||
expiryTime := time.Second * time.Duration(rcf.sovConfig.OutgoingSubscribedEvents.TimeToWaitForUnconfirmedOutGoingOperationInSeconds) | |||
|
|||
err = rtc.extraHeaderSigVerifierHandler.RegisterExtraHeaderSigVerifier(rcf.extraVerifier) | |||
sovHeaderSigVerifier, err := headerCheck.NewSovereignHeaderSigVerifier(rcf.cryptoComponents.BlockSigner()) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The name of the variable was wrong and confused me;
extraHeaderSigVerifierHandler -> this is a holder, not a handler, please rename it
@@ -544,16 +544,16 @@ func (mrc *managedRunTypeComponents) TxPreProcessorCreator() preprocess.TxPrePro | |||
return mrc.runTypeComponents.txPreProcessorCreator | |||
} | |||
|
|||
// ExtraHeaderSigVerifierHandler returns the extra header sig verifier handler | |||
func (mrc *managedRunTypeComponents) ExtraHeaderSigVerifierHandler() headerCheck.ExtraHeaderSigVerifierHolder { | |||
// ExtraHeaderSigVerifierHolder returns the extra header sig verifier handler |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
// ExtraHeaderSigVerifierHolder returns the extra header sig verifier handler | |
// ExtraHeaderSigVerifierHolder returns the extra header sig verifier holder |
…r-sov-runtype-comp-16
3552ab2
into
feat/refactor-runtype-components
Reasoning behind the pull request
Proposed changes
Testing procedure
Pre-requisites
Based on the Contributing Guidelines the PR author and the reviewers must check the following requirements are met:
feat
branch created?feat
branch merging, do all satellite projects have a proper tag insidego.mod
?