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

request-response: Connection and codec failures are never reported #4597

Closed
oblique opened this issue Oct 6, 2023 · 3 comments
Closed

request-response: Connection and codec failures are never reported #4597

oblique opened this issue Oct 6, 2023 · 3 comments

Comments

@oblique
Copy link
Contributor

oblique commented Oct 6, 2023

At Eiger we are facing an issue in celestia-node-rs: As soon as there is a connection error on the request-response protocol, our code gets stall because we were assuming that OutboundFailure would be generated. After investigation we noticed that the same happens if we return an error from Codec.

This is due to the fact that failures of upgrade_outbound stopped being reported by #3913. We also noticed that there is an ongoing effort of moving out of the upgrade infrastructure in #3914, which as we understood the issue will get fixed by it.

Is the wrapping the ConnectionHandler the only possible solution right now?

When do you expect to merge #3914? Is there anything pending? Is there any way we can help?

@thomaseizinger
Copy link
Contributor

When do you expect to merge #3914? Is there anything pending? Is there any way we can help?

It will be merged with the next breaking release which we are aiming to start next week or the week after.

@thomaseizinger
Copy link
Contributor

This is due to the fact that failures of upgrade_outbound stopped being reported by #3913

Hmm, that wasn't the intention. We should have probably returned the error and not just log it. If the current type signatures allow it, I am happy to merge and release a patch that forwards the error (but not by closing the connection).

@oblique
Copy link
Contributor Author

oblique commented May 29, 2024

Closing. This was fixed long ago.

@oblique oblique closed this as completed May 29, 2024
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

No branches or pull requests

2 participants