[PR #602] [MERGED] Adds ability to cap log size in redis #1252

New issue

Closed

opened 2026-03-04 03:07:38 +03:00 by kerem · 0 comments

kerem commented

2026-03-04 03:07:38 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/Seldaek/monolog/pull/602
Author: @ninjapenguin
Created: 7/12/2015
Status: ✅ Merged
Merged: 7/14/2015
Merged by: @Seldaek

Base: master ← Head: add-redis-apped-collection

📝 Commits (4)

0506942 Adds ability to cap log size in redis
4d59281 Fixes 5.3 compatibility in test cases
ae2b2d0 fixes reference for 5.3 in predis callback
12711d1 Switch back to using rpush

📊 Changes

2 files changed (+97 additions, -6 deletions)

View changed files

📝 src/Monolog/Handler/RedisHandler.php (+41 -6)
📝 tests/Monolog/Handler/RedisHandlerTest.php (+56 -0)

📄 Description

100% backward compatible, defaults to not capping collection size.

Additional constructor param of $capSize added which will ensure logs
list is treated as a FILO queue

Test coverage on new functionality

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/Seldaek/monolog/pull/602 **Author:** [@ninjapenguin](https://github.com/ninjapenguin) **Created:** 7/12/2015 **Status:** ✅ Merged **Merged:** 7/14/2015 **Merged by:** [@Seldaek](https://github.com/Seldaek) **Base:** `master` ← **Head:** `add-redis-apped-collection` --- ### 📝 Commits (4) - [`0506942`](https://github.com/Seldaek/monolog/commit/050694291e38ca010f577a45c98c8f9e4866499e) Adds ability to cap log size in redis - [`4d59281`](https://github.com/Seldaek/monolog/commit/4d592818685a2b1cae95d661180ce29de5f70379) Fixes 5.3 compatibility in test cases - [`ae2b2d0`](https://github.com/Seldaek/monolog/commit/ae2b2d0de16a968fcd72d2968dfbbff6ec89c89b) fixes reference for 5.3 in predis callback - [`12711d1`](https://github.com/Seldaek/monolog/commit/12711d133afd156e05080e5e1bedd3a281de99c4) Switch back to using rpush ### 📊 Changes **2 files changed** (+97 additions, -6 deletions) <details> <summary>View changed files</summary> 📝 `src/Monolog/Handler/RedisHandler.php` (+41 -6) 📝 `tests/Monolog/Handler/RedisHandlerTest.php` (+56 -0) </details> ### 📄 Description 100% backward compatible, defaults to not capping collection size. Additional constructor param of $capSize added which will ensure logs list is treated as a FILO queue Test coverage on new functionality --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>