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 robust logging support to the Filter class #12

Merged
merged 1 commit into from
Apr 23, 2024

Conversation

Thavarshan
Copy link
Owner

Overview

This Pull Request introduces a logging mechanism into the Filter class, enhancing debugging and operational monitoring capabilities. This feature allows developers to enable or disable logging at runtime, providing valuable insights into the filter application process, which is critical for both development and production environments.

Details

The modifications include:

  • Integration of the Psr\Log\LoggerInterface to support flexible logging strategies.
  • Methods to enable and disable logging, which helps control logging behavior dynamically.
  • Conditional logging throughout the filter processing methods to record key actions and decisions, enhancing traceability and troubleshooting.
  • Unit tests to ensure the logging functionality works as expected under various conditions.

Implications

With the introduction of logging:

  • Developers can now track the filter application process more transparently.
  • It becomes easier to diagnose issues and understand the behavior of the Filter class in complex scenarios.
  • The feature is optional and can be turned on or off, ensuring that performance is not impacted when logging is not required.

@Thavarshan Thavarshan added documentation Improvements or additions to documentation enhancement New feature or request labels Apr 23, 2024
@Thavarshan Thavarshan self-assigned this Apr 23, 2024
@Thavarshan Thavarshan merged commit 5d16317 into main Apr 23, 2024
6 checks passed
@Thavarshan Thavarshan deleted the feature/logging-support branch April 23, 2024 01:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant