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

Upstream connection events, EOF and I/O errors are never propagated to network filters #13940

Closed
PiotrSikora opened this issue Nov 9, 2020 · 5 comments
Labels
area/connection bug stale stalebot believes this issue/PR has not been touched recently

Comments

@PiotrSikora
Copy link
Contributor

The read/write network filters never receive information about:

which significantly limits robustness and capabilities of WebAssembly and ext_proc extensions.

cc @mattklein123 @gbrail @kyessenov

@PiotrSikora PiotrSikora added bug triage Issue requires triage labels Nov 9, 2020
@kyessenov
Copy link
Contributor

Thanks for digging into it. I wonder if it is possible to solve this fully for raw byte streams without application-level semantics on framing. I think there's some just some inherent non-determinism in closure with TIME_WAIT and proxy in the middle.

@antoniovicente
Copy link
Contributor

What actions would wasm filters want to take of midstream errors or EOF?

I assume that there's also a logging issue; are HTTP requests that fail after partial response tagged appropriately?

@PiotrSikora
Copy link
Contributor Author

Because upstream connection events are not propagated to network filters, Wasm is currently using end_stream=true (which is missing in case of mid-stream error) as a signal for upstream connection close events.

More generally, I'd like to be able to log that the connection was prematurely closed from within Wasm plugins.

@github-actions
Copy link

This issue has been automatically marked as stale because it has not had activity in the last 30 days. It will be closed in the next 7 days unless it is tagged "help wanted" or "no stalebot" or other activity occurs. Thank you for your contributions.

@github-actions github-actions bot added the stale stalebot believes this issue/PR has not been touched recently label Dec 14, 2020
@github-actions
Copy link

This issue has been automatically closed because it has not had activity in the last 37 days. If this issue is still valid, please ping a maintainer and ask them to label it as "help wanted" or "no stalebot". Thank you for your contributions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/connection bug stale stalebot believes this issue/PR has not been touched recently
Projects
None yet
Development

No branches or pull requests

4 participants