Skip to content
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

Relayer: Test that relayer fails and prevents tx submition if the message is not ready for relay #1993

Open
GabrielBuragev opened this issue Jun 29, 2023 · 0 comments
Labels

Comments

@GabrielBuragev
Copy link
Contributor

GabrielBuragev commented Jun 29, 2023

The OP relayer service lacked a check for the READY_FOR_RELAY message status. With the OP Bedrock work, we somehow removed it.
Without that, there was a possibility that the relayer tries to relay a message even though the message haven't arrived on L1 - which results in failing transactions on L1.

This was fixed in #1992 but it would be good to write some tests for it and in general think about how we can extend the tests for the relayer.

@GabrielBuragev GabrielBuragev added bug Something isn't working test env and removed bug Something isn't working labels Jun 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant