Skip to content

Revert "dynamic_modules: add support for disabling HTTP per-route filter (#42766)"#42771

Merged
wbpcode merged 1 commit intoenvoyproxy:mainfrom
wbpcode:dev-revert-a-api-change
Dec 25, 2025
Merged

Revert "dynamic_modules: add support for disabling HTTP per-route filter (#42766)"#42771
wbpcode merged 1 commit intoenvoyproxy:mainfrom
wbpcode:dev-revert-a-api-change

Conversation

@wbpcode
Copy link
Copy Markdown
Member

@wbpcode wbpcode commented Dec 25, 2025

Commit Message: Revert "dynamic_modules: add support for disabling HTTP per-route filter (#42766)"
Additional Description:

We have provided common disabled flag in the route configuration and this API may be unnecessary (and new api shepherd reviewed it. We need to revisit it before put it into our main.

Risk Level: low.
Testing: n/a.
Docs Changes: n/a.
Release Notes: n/a.
Platform Specific Features: n/a.

@repokitteh-read-only
Copy link
Copy Markdown

CC @envoyproxy/api-shepherds: Your approval is needed for changes made to (api/envoy/|docs/root/api-docs/).
envoyproxy/api-shepherds assignee is @adisuissa
CC @envoyproxy/api-watchers: FYI only for changes made to (api/envoy/|docs/root/api-docs/).

🐱

Caused by: #42771 was opened by wbpcode.

see: more, trace.

@wbpcode wbpcode merged commit cfcffd0 into envoyproxy:main Dec 25, 2025
25 of 26 checks passed
@wbpcode wbpcode deleted the dev-revert-a-api-change branch December 25, 2025 03:51
grnmeira pushed a commit to grnmeira/envoy that referenced this pull request Mar 20, 2026
…ter (envoyproxy#42766)" (envoyproxy#42771)

Commit Message: Revert "dynamic_modules: add support for disabling HTTP
per-route filter (envoyproxy#42766)"
Additional Description:

We have provided common disabled flag in the route configuration and
this API may be unnecessary (and new api shepherd reviewed it. We need
to revisit it before put int into our main.

Risk Level: low.
Testing: n/a.
Docs Changes: n/a.
Release Notes: n/a.
Platform Specific Features: n/a.

Signed-off-by: Gustavo <grnmeira@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants