Skip to content

feat: resend message #259

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

Merged
merged 2 commits into from
Apr 9, 2025
Merged

Conversation

0xDiscotech
Copy link
Contributor

@0xDiscotech 0xDiscotech commented Apr 8, 2025

Description

Introducing a new resendMessage to re-emit the SentMessage event for a message that was never relayed and became very old for the supervisor to pick it up.

Metadata

Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

Comments suppressed due to low confidence (1)

protocol/resend-messages.md:1

  • [nitpick] Consider replacing the placeholder '[Project Name]' with the actual project name for clarity.
# [Project Name]: Design Doc

Comment on lines 35 to 36
A hash collision could occur where different message params result in any valid message sent. This would involve a bug in the `Hashing` library logic or a hash collision, for which it’s likelihood is very low.

Copy link
Preview

Copilot AI Apr 8, 2025

Choose a reason for hiding this comment

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

[nitpick] Consider correcting the contraction 'it’s' to 'its' and rephrasing the sentence for improved clarity.

Suggested change
A hash collision could occur where different message params result in any valid message sent. This would involve a bug in the `Hashing` library logic or a hash collision, for which it’s likelihood is very low.
A hash collision could occur where different message parameters result in a valid message being sent. This would involve a bug in the `Hashing` library logic or a hash collision, for which its likelihood is very low.

Copilot uses AI. Check for mistakes.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@tynes tynes merged commit 8f05ed5 into ethereum-optimism:main Apr 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants