Skip to content

new generic proxy extension to contrib#18765

Closed
wbpcode wants to merge 3 commits intoenvoyproxy:mainfrom
wbpcode:generic_proxy
Closed

new generic proxy extension to contrib#18765
wbpcode wants to merge 3 commits intoenvoyproxy:mainfrom
wbpcode:generic_proxy

Conversation

@wbpcode
Copy link
Copy Markdown
Member

@wbpcode wbpcode commented Oct 26, 2021

Signed-off-by: wbpcode wbphub@live.com

Commit Message: new generic proxy extension to contrib

Additional Description:

Ref #18209 for more background information.

This PR is under active development. It's only used to make the maintainers and sponsor can give a quick review to the key API or interface.

This is a huge feature, so I chose to add it to contrib instead of the main extension repository. If its value and stability are verified later, we can consider moving it to the main extension repository.

Risk Level: N/A.
Testing: N/A.
Docs Changes: N/A.
Release Notes: N/A.
Platform Specific Features: N/A.

Signed-off-by: wbpcode <comems@msn.com>
@wbpcode
Copy link
Copy Markdown
Member Author

wbpcode commented Oct 26, 2021

cc @rojkov

Hi, @rojkov, here are some most important interfaces. Can you give it some comments to help me take this work to the next step . Because I am not sure that I fully understand what you said HW friendly.

codec:

contrib/generic_proxy/filters/network/source/interface/generic_codec.h

stream:

contrib/generic_proxy/filters/network/source/interface/generic_stream.h

@wbpcode wbpcode marked this pull request as draft October 26, 2021 08:29
wbpcode added 2 commits October 26, 2021 09:34
Signed-off-by: wbpcode <comems@msn.com>
Signed-off-by: wbpcode <comems@msn.com>
@zhaohuabing
Copy link
Copy Markdown
Member

#18823 has been created to represent the merged design.

@wbpcode
Copy link
Copy Markdown
Member Author

wbpcode commented Nov 10, 2021

new PR: #18944

@wbpcode wbpcode closed this Nov 10, 2021
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