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

mobile: Fix the debug log message in the network configuration filter #34093

Merged
merged 1 commit into from May 10, 2024
Merged

mobile: Fix the debug log message in the network configuration filter #34093

merged 1 commit into from May 10, 2024

Conversation

fredyw
Copy link
Member

@fredyw fredyw commented May 10, 2024

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

@fredyw
Fix the debug log message
f64b579 
Signed-off-by: Fredy Wijaya <fredyw@google.com>
@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: #34093 was opened by fredyw.

see: more, trace.

@fredyw fredyw marked this pull request as ready for review May 10, 2024 19:52
@fredyw
Copy link
Member Author

fredyw commented May 10, 2024

/assign @abeyad

@fredyw fredyw enabled auto-merge (squash) May 10, 2024 19:52
@fredyw fredyw merged commit 4b0495b into envoyproxy:main May 10, 2024
35 checks passed
@fredyw fredyw deleted the network_config_filter_debug branch May 10, 2024 20:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@abeyad abeyad abeyad approved these changes

@RyanTheOptimist RyanTheOptimist Awaiting requested review from RyanTheOptimist RyanTheOptimist is a code owner

@alyssawilk alyssawilk Awaiting requested review from alyssawilk alyssawilk is a code owner

Assignees

@abeyad abeyad

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@fredyw @abeyad