Skip to content

Conversation

keyolk
Copy link
Contributor

@keyolk keyolk commented Jul 30, 2025

It is about to fix #38585 again

@keyolk keyolk requested a review from mattklein123 as a code owner July 30, 2025 13:58
Copy link

Hi @keyolk, welcome and thank you for your contribution.

We will try to review your Pull Request as quickly as possible.

In the meantime, please take a look at the contribution guidelines if you have not done so already.

🐱

Caused by: #40488 was opened by keyolk.

see: more, trace.

@keyolk keyolk force-pushed the fix-redis-segfault-rebased branch from 1e4234d to 5811f1e Compare July 30, 2025 14:32
@keyolk keyolk force-pushed the fix-redis-segfault-rebased branch from 5811f1e to 73d5b0b Compare July 30, 2025 15:03
- Remove trailing whitespace in test file
- Fix line wrapping in constructor initialization
- Apply consistent formatting per project standards

Signed-off-by: Chanhun Jeong <[email protected]>
@keyolk keyolk force-pushed the fix-redis-segfault-rebased branch from 73d5b0b to e4d0cfd Compare July 31, 2025 00:25
@mattklein123 mattklein123 merged commit ec228f8 into envoyproxy:main Aug 5, 2025
22 checks passed
eziskind pushed a commit to eziskind/envoy that referenced this pull request Aug 8, 2025
It is about to fix envoyproxy#38585
again

---------

Signed-off-by: Chanhun Jeong <[email protected]>
Signed-off-by: Elisha Ziskind <[email protected]>
VivekSubr pushed a commit to VivekSubr/envoy-dynamic-modules-cpp-sdk that referenced this pull request Sep 16, 2025
It is about to fix envoyproxy#38585
again

---------

Signed-off-by: Chanhun Jeong <[email protected]>
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.

Got segfault with redis proxy cluster
2 participants