-
Notifications
You must be signed in to change notification settings - Fork 79
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
cmd/createdkg: verify enr in create dkg #757
Conversation
Codecov Report
@@ Coverage Diff @@
## main #757 +/- ##
==========================================
- Coverage 54.57% 54.38% -0.19%
==========================================
Files 107 107
Lines 10478 10485 +7
==========================================
- Hits 5718 5702 -16
- Misses 3919 3936 +17
- Partials 841 847 +6
Continue to review full report at Codecov.
|
cmd/createdkg.go
Outdated
for i, opENR := range conf.OperatorENRs { | ||
_, err := p2p.DecodeENR(opENR) | ||
if err != nil { | ||
return errors.Wrap(err, fmt.Sprintf("invalid ENR of operator %d", i)) |
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.
you should just be able to use z.Int("index",i))
instead of fmt.Sprintf
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.
done
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.
One or two small tests would be awesome
Verifies operator ENRs to be in standard form at the start of create dkg. Also make frost dkg as the default one.
category: refactor
ticket: #752