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.

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

Add new filter to disable query logging: ep_disable_query_logging #217

Merged
merged 1 commit into from
Dec 3, 2024

Conversation

rebeccahum
Copy link

@rebeccahum rebeccahum commented Nov 27, 2024

Description

Pulls in 10up#4019

Checklist

Please make sure the items below have been covered before requesting a review:

  • This change works and has been tested locally (or has an appropriate fallback).
  • This change works and has been tested on a Go sandbox.
  • This change has relevant unit tests (if applicable).
  • This change has relevant documentation additions / updates (if applicable).
  • This change has the fix PRed upstream (if applicable). If not applicable, it has the relevant "// VIP: reason for the discrepancy with upstream" comment in places where the code is discrepant.

Steps to Test

@rebeccahum rebeccahum marked this pull request as ready for review November 27, 2024 20:45
@rebeccahum rebeccahum added the [Status] Upstream fix If this has been fixed upstream or in progress to be. label Nov 27, 2024
Copy link

@katag9k katag9k left a comment

Choose a reason for hiding this comment

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

LGTM

@rebeccahum rebeccahum merged commit a66065a into develop Dec 3, 2024
3 checks passed
@rebeccahum rebeccahum deleted the pr/4019 branch December 3, 2024 18:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Status] Upstream fix If this has been fixed upstream or in progress to be.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants