-
Notifications
You must be signed in to change notification settings - Fork 197
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
Peer authentication sender #3771
Conversation
iulianpascalau
commented
Feb 8, 2022
- added peer authentication sender
- added routineHandler & timerWrapper sub-components
- renamed some interfaces, stub implementations and errors
- added routineHandler & timerWrapper sub-components - renamed some interfaces, stub implementations and errors
pkBytes, _ := sender.publicKey.ToByteArray() | ||
log.Info("args", "pid", args.Messenger.ID().Pretty(), "bls sk", skBytes, "bls pk", pkBytes) | ||
|
||
// verify the received bytes if they can be converted in a valid peer authentication message |
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 can transform these comments into t.Run("description")
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.
can be done, sure
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
|
||
const minTimeBetweenSends = time.Second | ||
|
||
// ArgPeerAuthenticationSender represents the arguments for the heartbeat sender |
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.
// ArgPeerAuthenticationSender represents the arguments for the heartbeat sender | |
// ArgPeerAuthenticationSender represents the arguments for the peer authentication sender |
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