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

dynamic_modules: adds on_http_filter_* event hooks #37325

Open
wants to merge 15 commits into
base: main
Choose a base branch
from

Conversation

mathetake
Copy link
Member

@mathetake mathetake commented Nov 22, 2024

Commit Message: dynamic_modules: adds `on_http_filter_* event hooks
Additional Description:

This implements HTTP filter event hooks for dynamic modules.
More precisely, envoy_dynamic_module_on_http_filter_* are added to the ABI,
and they are integrated with the HTTP filter implementation in Envoy side.

The most change here is almost mechanical despite the size of changes.

Risk Level: low
Testing: done
Docs Changes: n/a
Release Notes: n/a
Platform Specific Features: n/a
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

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: #37325 was opened by mathetake.

see: more, trace.

@mathetake mathetake changed the title dynamic_modules: implements HTTP filter event hooks dynamic_modules: adds on_http_filter_* event hooks Nov 22, 2024
@mathetake mathetake marked this pull request as ready for review November 27, 2024 19:45
@mathetake
Copy link
Member Author

cc @marc-barry

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.

1 participant