-
Notifications
You must be signed in to change notification settings - Fork 948
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
Grarco/replay protection delay write REVIEW ONLY #2108
Grarco/replay protection delay write REVIEW ONLY #2108
Conversation
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.
Only a minor comment, but @tzemanovic should also review this.
protocol::Error::ReplayAttempt(_), | ||
) = msg | ||
{ | ||
// Remove the wrapper hash but keep the inner tx |
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 we further explicate this comment to explain why this is safe? (as I understand it, because the inner transaction hash will still be in storage, and block any future replay)
8cbba5c
to
b9e4b7a
Compare
1e91762
to
12aeb34
Compare
b9e4b7a
to
c0e14eb
Compare
12aeb34
to
1878a4c
Compare
c0e14eb
to
6c1d784
Compare
REVIEW ONLY