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.

Sign up for GitHub

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

Jump to bottom

listener: more exception cleanup #36198

Merged
merged 1 commit into from
Sep 24, 2024
Merged

listener: more exception cleanup #36198

merged 1 commit into from
Sep 24, 2024

Conversation

alyssawilk
Copy link
Contributor

@alyssawilk alyssawilk commented Sep 18, 2024 •
edited
Loading

Risk Level: low
Testing: updated tests
Docs Changes: n/a
Release Notes: n/a
envoyproxy/envoy-mobile#176

@repokitteh-read-only RepoKitteh - Read Only
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: #36198 was opened by alyssawilk.

see: more, trace.

@alyssawilk
listener: more exception cleanup
8718afd 
Signed-off-by: Alyssa Wilk <alyssar@chromium.org>
@alyssawilk alyssawilk marked this pull request as ready for review September 23, 2024 17:27
@alyssawilk alyssawilk enabled auto-merge (squash) September 23, 2024 18:07
@alyssawilk alyssawilk merged commit a141498 into envoyproxy:main Sep 24, 2024
40 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ggreenway ggreenway ggreenway approved these changes

Assignees

@ggreenway ggreenway

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@alyssawilk @ggreenway