Skip to content

889116 - Make log optional + zip log file when size exceed retention_size#20

Open
pngouahbeapi wants to merge 1 commit into
masterfrom
issue/88916
Open

889116 - Make log optional + zip log file when size exceed retention_size#20
pngouahbeapi wants to merge 1 commit into
masterfrom
issue/88916

Conversation

@pngouahbeapi
Copy link
Copy Markdown
Contributor

@pngouahbeapi pngouahbeapi commented May 13, 2026

Note

Medium Risk
Touches cron-time filesystem logging behavior (enable/disable + rotation/compression), which could impact debugging visibility and file/permission expectations in production.

Overview
Adds bea_sender_is_file_logging_enabled() (configurable via BEA_SENDER_FILE_LOGGING or bea_sender_file_logging_enabled) and wires it into the sender and bounce cron loggers so file logging can be turned off.

Updates Bea_Log to accept a write-enabled flag (making logging a no-op when disabled) and, on rotation, optionally compresses rotated log files into .zip archives when ZipArchive is available; bumps plugin version to 1.4.5 and documents the change in the changelog.

Reviewed by Cursor Bugbot for commit 12d897b. Bugbot is set up for automated code reviews on this repo. Configure here.

@francoistibo francoistibo requested a review from Rahe May 13, 2026 10:11
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.

2 participants