Description
Logman is a module which provides a better UI for searching dblog based watchdog messages and includes apache access log search.
The UI is intended to provide easy search of log messages with more information. The UI can be used to search both the watchdog and apache access log messages.The watchdog search tool provides searching ability on almost all fields of watchdog table. Similarly apache access log search tool provides searching facility on most of the common fields.
Module provides a UI on every page indicating count of number of dblog based watchdog messages grouped by severity.
Module also provides a statistics page with google column charts on dblog based watchdog and apache access log data.
Summary of features provided.
- Watchdog based log search
- Apache access log search
- Statistics page based on apache access log and watchdog log
- Page UI that appear on each (only for users with permission and can be disabled), as an alert for number of watchdog based log messages for that page grouped by severity.
This version of the module provides search and statistics facilities for dblog based watchdog messages, the future versions are intended to support gelf based watchdog messages.
Other similar modules and how Logman is different
Following are links to some the similar modules:
https://drupal.org/project/views_watchdog
https://drupal.org/project/logging_alerts
https://drupal.org/project/watchdog_filtering
https://drupal.org/project/better_watchdog_ui
The views_watchdog and logging_alerts are available with D6 core, but views_watchdog provides a basic watchdog log browsing whereas logman watchdog search tool will provide extensive search on log messages. The logging_alerts is entirely different from logman as it provides email and web server based logging.
The watchdog_fitering and better_watchdog_ui is only available with D7 core. Still comparing with logman watchdog_filtering provides again basic kind of log search and better_watchdog_ui does provides better watchdog log search, however the logman is still different from these two modules as well because it not only provides extensive search on watchdog log messages but also includes features like apache access log search, statistics and UI on each page as an alert based on watchdog log.
Link to repository
git clone --branch 6.x-1.x http://git.drupal.org/sandbox/AbhijeetGhosh/2030383.git logman
Contributions with other projects
https://drupal.org/node/1957158 (Drupal 8 core)
https://drupal.org/node/1996644 (Drupal 8 core)
Reviews of project
https://drupal.org/node/2031701#comment-7633121
https://drupal.org/node/2088431#comment-7873571
https://drupal.org/node/2067533#comment-7874309
Reviews of other project:
https://drupal.org/node/2089695#comment-7883575
https://drupal.org/node/2094477#comment-7883681
Comments
Comment #0.0
abghosh82 CreditAttribution: abghosh82 commentedUpdated the link in reviews for project.
Comment #0.1
abghosh82 CreditAttribution: abghosh82 commentedAdded the link to repository.
Comment #0.2
abghosh82 CreditAttribution: abghosh82 commentedUpdated the link to repository.
Comment #0.3
abghosh82 CreditAttribution: abghosh82 commentedUpdated the description.
Comment #1
abghosh82 CreditAttribution: abghosh82 commentedPlease review my module. Setting it for review.
Comment #2
PA robot CreditAttribution: PA robot commentedThere are some errors reported by automated review tools, did you already check them? See http://pareview.sh/pareview/httpgitdrupalorgsandboxAbhijeetGhosh2030383git
We 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
abghosh82 CreditAttribution: abghosh82 commentedI have fixed the Drupal CS review issues. Setting to review again.
Comment #4
alexmoreno CreditAttribution: alexmoreno commentedHi my friend,
Help with some reviews, 3 at least and put yourself the bonus review tag so other developers will review type module.
Thank you
Comment #5
t14 CreditAttribution: t14 commentedHi
I installed this module and took it for a test drive.
I like the idea. Below are the following issues I came across
Error message on page admin/reports/logman
Notice: Undefined offset: 14 in LogmanApacheSearch->readApacheLog() (line 192 of /sites/all/modules/logman/2030383/includes/lib/LogmanApacheSearch.php).
You need to create a hook_uninstall in your .install file and use variable_del. So that your users have the option to remove all of the variables stored in the databases by your module.
Hope that was helpful
T
Comment #6
abghosh82 CreditAttribution: abghosh82 commentedThanks for review t14, I agree should have added hook_uninstall(). Fixed the couple of issues you mentioned.
Comment #6.0
abghosh82 CreditAttribution: abghosh82 commentedUpdated the description.
Comment #6.1
abghosh82 CreditAttribution: abghosh82 commentedUpdated reviews of project.
Comment #6.2
abghosh82 CreditAttribution: abghosh82 commentedAdded contributions with other projects.
Comment #7
abghosh82 CreditAttribution: abghosh82 commentedI have done 3 manual reviews so adding a review bonus tag.
Comment #8
klausiReview of the 6.x-1.x branch:
This automated report was generated with PAReview.sh, your friendly project application review script. You can also use the online version to check your project. You have to get a review bonus to get a review from me.
manual review:
Otherwise looks RTBC to me. Removing review bonus tag, you can add it again if you have done another 3 reviews of other projects.
Assigning to kscheirer as he might have time to take a final look at this.
Comment #9
abghosh82 CreditAttribution: abghosh82 commentedThanks a lot klausi for your review and a sort of positive feedback for RTBC.
For the PAReview issues not sure, can theme_table and theme_page not be used, which I thought was a valid drupal 6 function?
As I have mentioned in issue description this module is different to others, since in this module there is apache access log search and statistics for both watchdog and apache access log and statistics/alert on each page, the part which is not there in any on the other similar modules, also planning to support gelf in this module, so would like to keep this separate.
For logman_init(), I have used this for the page statistics/alert that is visible at the bottom of each page. The statistics UI sits on the hook_footer (there is no menu callback for it) where I cannot use drupal_add_css or drupal_add_js, as I think it takes affect only in menu callback or hook_init. Also I have added the visibility for this purpose only, so if user chooses not to display the UI, thus the UI and corresponding js and css will not be added to that page. Let me know how should I use it in this case.
Fixed the doc block in logman_statistics_form().
Fixed the comment issues in LogmanSearchInterface.
For LogmanWatchdogSearch sql queries vulnerable to SQL injection, I went to the link you suggested, and checked the code I have used database abstraction and have used substitutions for all fields except timestamps, which I think should get filtered out with the use of strtotime in calling function in logman_watchdog.inc. For statistics I have not used substitutions but the value which it depends on is drupal site's page URI and timestamps generated by the code. Please suggest me how would you like me to fix this.
Fixed the php notice.
Comment #9.0
abghosh82 CreditAttribution: abghosh82 commentedUpdated reviews of project.
Comment #9.1
abghosh82 CreditAttribution: abghosh82 commentedUpdated reviews of other project.
Comment #10
kscheirerThanks for addressing the similar modules on your project page.
date_default_timezone_set("Europe/London");
?theme('pager')
instead of directly astheme_pager()
. This is done so overrides have a chance to intercept those calls. If you call theme_pager directly, no override can happen.Very nicely written module!
Thanks for your contribution, abghosh82!
I updated your account to let you 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 lots more contributors chatting on IRC in #drupal-contribute. So, come hang out and get involved!
Thanks, 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 to the dedicated reviewer(s) as well.
----
Top Shelf Modules - Crafted, Curated, Contributed.
Comment #11
abghosh82 CreditAttribution: abghosh82 commentedThanks a lot kscheirer, for your review and giving me RTBC.
I would fix the issues you and klausi have mentioned and also thanks for explaining me the difference between theme_pager and theme('pager'). I would fix this as well.
Setting default timezone may not be required, I might remove it, after doing some tests.
Yes I am planning a D7 version of this module with gelf support, I will also enhance this D6 version with gelf support.
Comment #12
abghosh82 CreditAttribution: abghosh82 commentedFix the theme_table and theme_pager issues, and made SQL queries more secure. Removed
date_default_timezone_set("Europe/London");
as not required.Now I will release this module to full project.
Thanks for all reviews.
Comment #13.0
(not verified) CreditAttribution: commentedAdded one more project review.