Skip to content

Revert "response decoder: add new interfaces for life time tracking o…#41141

Merged
botengyao merged 1 commit intoenvoyproxy:mainfrom
danzh2010:reverthandle
Sep 19, 2025
Merged

Revert "response decoder: add new interfaces for life time tracking o…#41141
botengyao merged 1 commit intoenvoyproxy:mainfrom
danzh2010:reverthandle

Conversation

@danzh2010
Copy link
Copy Markdown
Contributor

…f response decoder (#41048)"

This reverts commit 566b192.

It's causing crash at ResponseDecoderWrapper

…f response decoder (envoyproxy#41048)"

This reverts commit 566b192.

Signed-off-by: Dan Zhang <danzh@google.com>
@repokitteh-read-only
Copy link
Copy Markdown

CC @envoyproxy/runtime-guard-changes: FYI only for changes made to (source/common/runtime/runtime_features.cc).

🐱

Caused by: #41141 was opened by danzh2010.

see: more, trace.

@danzh2010
Copy link
Copy Markdown
Contributor Author

/assign @botengyao

Copy link
Copy Markdown
Member

@botengyao botengyao left a comment

Choose a reason for hiding this comment

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

thanks!

@botengyao botengyao enabled auto-merge (squash) September 19, 2025 15:20
@botengyao botengyao merged commit c111d5e into envoyproxy:main Sep 19, 2025
24 checks passed
mbadov pushed a commit to mbadov/envoy that referenced this pull request Sep 22, 2025
envoyproxy#41141)

…f response decoder (envoyproxy#41048)"

This reverts commit 566b192.

It's causing crash at ResponseDecoderWrapper

Signed-off-by: Dan Zhang <danzh@google.com>
Co-authored-by: Dan Zhang <danzh@google.com>
Signed-off-by: Misha Badov <mbadov@google.com>
lucaschimweg pushed a commit to lucaschimweg/envoy that referenced this pull request Sep 23, 2025
envoyproxy#41141)

…f response decoder (envoyproxy#41048)"

This reverts commit 566b192.

It's causing crash at ResponseDecoderWrapper

Signed-off-by: Dan Zhang <danzh@google.com>
Co-authored-by: Dan Zhang <danzh@google.com>
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