starred/monolog
1
0
Fork 0
mirror of https://github.com/Seldaek/monolog.git synced 2026-04-26 16:15:49 +03:00
Code Issues 31 Projects Releases 5 Packages Wiki Activity

[GH-ISSUE #312] HipChatHandler doesn't implement handleBatch() #103

New issue
Closed
opened 2026-03-04 02:12:14 +03:00 by kerem · 3 comments
kerem commented 2026-03-04 02:12:14 +03:00
Owner
Copy link

Originally created by @leevigraham on GitHub (Jan 23, 2014).
Original GitHub issue: https://github.com/Seldaek/monolog/issues/312

Could this be implemented?

HipChat Api supports max 10k characters per post which could be an issue. To solve this we could concatenate multiple messages checking if the result would be more or less than 10k characters.

My other concern is how to handle a single message over 10k characters. This probably hasn't been an issue given a single log message is generally quite short.

Originally created by @leevigraham on GitHub (Jan 23, 2014). Original GitHub issue: https://github.com/Seldaek/monolog/issues/312 Could this be implemented? HipChat Api supports max 10k characters per post which could be an issue. To solve this we could concatenate multiple messages checking if the result would be more or less than 10k characters. My other concern is how to handle a single message over 10k characters. This probably hasn't been an issue given a single log message is generally quite short.
kerem closed this issue 2026-03-04 02:12:14 +03:00
kerem commented 2026-03-04 02:12:15 +03:00
Author
Owner
Copy link

@stof commented on GitHub (Jan 23, 2014):

handleBatch is implemented: https://github.com/Seldaek/monolog/blob/master/src/Monolog/Handler/HipChatHandler.php#L165

It does not seem to handle the max length though

<!-- gh-comment-id:33118719 --> @stof commented on GitHub (Jan 23, 2014): `handleBatch` is implemented: https://github.com/Seldaek/monolog/blob/master/src/Monolog/Handler/HipChatHandler.php#L165 It does not seem to handle the max length though
kerem commented 2026-03-04 02:12:15 +03:00
Author
Owner
Copy link

@leevigraham commented on GitHub (Jan 23, 2014):

My bad. I just updated the bundle and obviously didn't check the actual Seldaek/monolog repo.

<!-- gh-comment-id:33119240 --> @leevigraham commented on GitHub (Jan 23, 2014): My bad. I just updated the bundle and obviously didn't check the actual Seldaek/monolog repo.
kerem commented 2026-03-04 02:12:15 +03:00
Author
Owner
Copy link

@leevigraham commented on GitHub (Jan 23, 2014):

Added limit to handleBatch calls: PR #313

<!-- gh-comment-id:33123858 --> @leevigraham commented on GitHub (Jan 23, 2014): Added limit to handleBatch calls: PR #313
Sign in to join this conversation.
No Branch/Tag specified
Branches Tags
main
esfix
2.x
1.x
3.10.0
2.11.0
3.9.0
3.8.1
3.8.0
2.10.0
3.7.0
3.6.0
2.9.3
3.5.0
2.9.2
3.4.0
3.3.1
2.9.1
3.3.0
2.9.0
3.2.0
2.8.0
3.1.0
2.7.0
1.27.1
3.0.0
2.6.0
3.0.0-RC1
2.5.0
2.4.0
1.27.0
2.3.5
2.3.4
2.3.3
2.3.2
2.3.1
2.3.0
1.26.1
2.2.0
1.26.0
2.1.1
1.25.5
2.1.0
1.25.4
2.0.2
1.25.3
2.0.1
1.25.2
1.25.1
1.25.0
2.0.0
2.0.0-beta2
2.0.0-beta1
1.24.0
1.23.0
1.22.1
1.22.0
1.21.0
1.20.0
1.19.0
1.18.2
1.18.1
1.18.0
1.17.2
1.17.1
1.17.0
1.16.0
1.15.0
1.14.0
1.13.1
1.13.0
1.12.0
1.11.0
1.10.0
1.9.1
1.9.0
1.8.0
1.7.0
1.6.0
1.5.0
1.4.1
1.4.0
1.3.1
1.3.0
1.2.1
1.2.0
1.1.0
1.0.2
1.0.1
1.0.0
1.0.0-RC1
Labels
Clear labels   
Bug

   
Documentation

   
Feature

   
Needs Work

   
Support

   
pull-request

Mirrored from GitHub Pull Request

No labels
Bug
Documentation
Feature
Needs Work
Support
pull-request
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/monolog#103
No description provided.