Skip to content

v1.25: Prevent recursion during premature reset check (#30270)#30291

Merged
phlax merged 2 commits intoenvoyproxy:release/v1.25from
yanavlasov:recursion-fix/v1.25
Oct 23, 2023
Merged

v1.25: Prevent recursion during premature reset check (#30270)#30291
phlax merged 2 commits intoenvoyproxy:release/v1.25from
yanavlasov:recursion-fix/v1.25

Conversation

@yanavlasov
Copy link
Copy Markdown
Contributor

Backport #30270 into v1.25

Prevent doConnectionClose to be called recursively when connection with active requests is disconnected due to premature reset check.

Signed-off-by: Yan Avlasov <yavlasov@google.com>
@phlax phlax added this to the 1.25.12 milestone Oct 18, 2023
@phlax phlax closed this Oct 18, 2023
@phlax phlax reopened this Oct 18, 2023
@phlax phlax enabled auto-merge (rebase) October 18, 2023 15:58
@phlax
Copy link
Copy Markdown
Member

phlax commented Oct 18, 2023

apologies i closed this by accident, kicking CI

Signed-off-by: Yan Avlasov <yavlasov@google.com>
Copy link
Copy Markdown
Member

@phlax phlax 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 @yanavlasov

@phlax
Copy link
Copy Markdown
Member

phlax commented Oct 23, 2023

/retest

@phlax phlax merged commit b1fb3d5 into envoyproxy:release/v1.25 Oct 23, 2023
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