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

Support ext_proc FULL_DUPLEX_STREAMED mode #388

Open
Tracked by #362
AndresGuedez opened this issue Feb 21, 2025 · 3 comments
Open
Tracked by #362

Support ext_proc FULL_DUPLEX_STREAMED mode #388

AndresGuedez opened this issue Feb 21, 2025 · 3 comments
Assignees

Comments

@AndresGuedez
Copy link

What would you like to be added:

Add support for FULL_DUPLEX_STREAMED mode.

See https://github.com/envoyproxy/envoy/blob/60aed1c0ae520ed4e1d828c998ae961b4314542c/api/envoy/extensions/filters/http/ext_proc/v3/processing_mode.proto#L82-L111.

Why is this needed:

FULL_DUPLEX_STREAMED ensures the extension is able to receive arbitrarily large request body payload and defer a header response until the parsing is completed.

@ahg-g
Copy link
Contributor

ahg-g commented Feb 21, 2025

/assign @liu-cong

@ahg-g
Copy link
Contributor

ahg-g commented Feb 24, 2025

/assign @kfswain

@kfswain
Copy link
Collaborator

kfswain commented Feb 25, 2025

Let's keep this assigned to me, but I'm going to hold on this for a short time. We have some other folks working on this, so some of the logic may be reusable.

I have a PR out to EGW to see if we can make configuring this a little easier (i.e. just use the EnvoyExtensionPolicy rather than having to patch the xDS config directly)

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

4 participants