Skip to content

Conversation

@agrawroh
Copy link
Member

@agrawroh agrawroh commented Dec 31, 2025

Description

This PR adds support for more methods in Dynamic Modules for Network Filters to make HTTP callouts.


Commit Message: dynamic_modules: add more methods to network filters ABI
Additional Description: Adds support for more methods in Dynamic Modules for Network Filters to make HTTP callouts.
Risk Level: Low
Testing: Added Tests
Docs Changes: Added
Release Notes: N/A

@repokitteh-read-only
Copy link

As a reminder, PRs marked as draft will not be automatically assigned reviewers,
or be handled by maintainer-oncall triage.

Please mark your PR as ready when you want it to be reviewed!

🐱

Caused by: #42815 was opened by agrawroh.

see: more, trace.

@agrawroh agrawroh force-pushed the feat-dm-ntwrk-6 branch 25 times, most recently from f10140d to 54d57f9 Compare January 3, 2026 09:13
@agrawroh agrawroh changed the title dynamic_modules: add more methods to listener filters ABI dynamic_modules: add more methods to network filters ABI Jan 3, 2026
@agrawroh agrawroh force-pushed the feat-dm-ntwrk-6 branch 2 times, most recently from b84da13 to 3012ada Compare January 3, 2026 17:12
@agrawroh agrawroh force-pushed the feat-dm-ntwrk-6 branch 12 times, most recently from d0ae9b3 to 0eac11f Compare January 5, 2026 03:19
@agrawroh agrawroh marked this pull request as ready for review January 5, 2026 03:44
Copy link
Member

@wbpcode wbpcode left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry, i forgot to submit my comments. And could you ensure all the ABI has the correct comments:)

Copy link
Member

@wbpcode wbpcode left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thanks.

@agrawroh agrawroh merged commit ab06c0a into envoyproxy:main Jan 6, 2026
25 checks passed
botengyao pushed a commit to botengyao/envoy that referenced this pull request Jan 6, 2026
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.

3 participants