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 #4019

Merged

Conversation

rebeccahum
Copy link
Contributor

Description of the Change

Adds a new filter called ep_disable_query_logging to disable query logging due to potential memory leaks.

Closes #4006

How to test the Change

add_filter( 'ep_disable_query_logging', '__return_true' );

Changelog Entry

Added - New filter ep_disable_query_logging to disable query logging

Credits

Props @davidsword

Checklist:

@rebeccahum rebeccahum mentioned this pull request Nov 26, 2024
1 task
@rebeccahum rebeccahum force-pushed the add/filter-ep_disable_query_logging branch from ef9f723 to e2e0571 Compare November 26, 2024 20:10
@felipeelia
Copy link
Member

Thanks @rebeccahum. Any chance you can add a docblock to that new filter following the pattern we have for the other filters?

@rebeccahum rebeccahum force-pushed the add/filter-ep_disable_query_logging branch 2 times, most recently from 7426929 to 0456bec Compare November 26, 2024 20:15
@rebeccahum rebeccahum force-pushed the add/filter-ep_disable_query_logging branch from 0456bec to 6f94523 Compare November 26, 2024 20:35
rebeccahum and others added 2 commits November 26, 2024 13:58
Co-authored-by: Felipe Elia <felipe.elia@gmail.com>
@rebeccahum rebeccahum force-pushed the add/filter-ep_disable_query_logging branch 2 times, most recently from 4d54c46 to 84541d9 Compare November 26, 2024 22:01
@rebeccahum rebeccahum force-pushed the add/filter-ep_disable_query_logging branch from 84541d9 to f0581f8 Compare November 26, 2024 22:13
@rebeccahum
Copy link
Contributor Author

@felipeelia Done! Thank you

@felipeelia felipeelia added this to the 5.2.0 milestone Nov 27, 2024
@felipeelia felipeelia merged commit 023712b into 10up:develop Nov 27, 2024
16 of 20 checks passed
@felipeelia felipeelia modified the milestones: 5.2.0, 5.1.4 Dec 3, 2024
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.

BUG: query logging memory hog
2 participants