External Logger
Synopsis
External Logger module for Drupal is a module that utilizes hook_watchdog() to catch all log events on a Drupal based site. And send the events to a compatible backend. The module is designed to center log messages of multiple environments and sites in a single location. All environments can be grouped to form a practical overview. Saving the developer the time of looking up multiple logs and/or multiple machines.
Features
Currently only houses external logging through the GELF-PHP library.
- GELF-PHP
- HTTP(S)
- UDP
Installation
- Clone or Copy+paste into modules directory
- Run "drush en external_logger -y"
- (Composer Manager should install the composer dependencies)
- Go to: /admin/config/system/external_logger and configure your server settings
- (Optional) Run "drush external_logger test-message" to check dependencies
Links
Git link: git clone --branch 7.x-0.x git://git.drupal.org/sandbox/Neodork/2553265.git
Project page: External Logger
Pareview result: http://pareview.sh/pareview/httpgitdrupalorgsandboxneodork2553265git
Preview message received: https://gyazo.com/a801e48bd50d489dade9ade469dbf95f
Comments
Comment #2
PA robot CreditAttribution: PA robot commentedWe are currently quite busy with all the project applications and we prefer projects with a review bonus. Please help reviewing and put yourself on the high priority list, then we will take a look at your project right away :-)
Also, you should get your friends, colleagues or other community members involved to review this application. Let them go through the review checklist and post a comment that sets this issue to "needs work" (they found some problems with the project) or "reviewed & tested by the community" (they found no major flaws).
I'm a robot and this is an automated message from Project Applications Scraper.
Comment #3
darol100 CreditAttribution: darol100 as a volunteer and commentedAutomated Review
No errors in the Pareview.sh - http://pareview.sh/pareview/httpgitdrupalorgsandboxneodork2553265git
Manual Review
For more information about how to fix this check function drupal_strtoupper page.
The starred items (*) are fairly big issues and warrant going back to Needs Work. Items marked with a plus sign (+) are important and should be addressed before a stable project release. The rest of the comments in the code walkthrough are recommendations.
If added, please don't remove the security tag, we keep that for statistics and to show examples of security problems.
I do not see any blocker, this is a well written project. Good job @Neodork.
This review uses the Project Application Review Template.
Comment #4
Neodork CreditAttribution: Neodork commentedHello darol100,
Thanks for taking your time reviewing my application.
Coding style & Drupal API usage
List of identified issues in no particular order. Use (*) and (+) to indicate an issue importance. Replace the text below by the issues themselves:
I shall look into it right awayfixed.Comment #5
apadernoThank you for your contribution!
I updated your account so you can promote this to a full project and also create new projects as either a sandbox or a "full" project.
Here are some recommended readings to help with excellent maintainership:
You can find more contributors chatting on IRC in #drupal-contribute. So, come hang out and stay involved!
Thank you, also, for your patience with the review process. Anyone is welcome to participate in the review process. Please consider reviewing other projects that are pending review. I encourage you to learn more about that process and join the group of reviewers.
Thanks go also to the dedicated reviewer(s) as well.