Bug #6187

Growl notifications emit PHP warnings if the specified name does not exist

Added by Phillip Davis over 3 years ago. Updated about 3 years ago.

Target version:
Start date:
Due date:
% Done:


Estimated time:
Affected Version:
Affected Architecture:


System->Advanced, Notifications
1) In the Growl section, "IP Address", put an invalid hostname (e.g. I put
2) Press "Test Growl Settings"
A bunch of warnings are emitted to the browser window like:
Warning: socket_sendto(): Host lookup failed [-10001]: Unknown host in /etc/inc/growl.class on line 78 Call Stack: 0.0001 239656 1. {main}() /usr/local/www/system_advanced_notifications.php:0 0.0197 808432 2. register_via_growl() /usr/local/www/system_advanced_notifications.php:202 0.2030 818272 3. Growl->register() /etc/inc/ 0.2031 818640 4. Growl->send() /etc/inc/growl.class:50 0.2031 818992 5. socket_sendto() /etc/inc/growl.class:78

A "crash" is reported on the dashboard like:
PHP Errors:
[17-Apr-2016 06:34:40 Etc/UTC] PHP Stack trace:
[17-Apr-2016 06:34:40 Etc/UTC] PHP 1. {main}() /usr/local/www/system_advanced_notifications.php:0
[17-Apr-2016 06:34:40 Etc/UTC] PHP 2. register_via_growl() /usr/local/www/system_advanced_notifications.php:202
[17-Apr-2016 06:34:40 Etc/UTC] PHP 3. Growl->register() /etc/inc/
[17-Apr-2016 06:34:40 Etc/UTC] PHP 4. Growl->send() /etc/inc/growl.class:50
[17-Apr-2016 06:34:40 Etc/UTC] PHP 5. socket_sendto() /etc/inc/growl.class:78

Forum discussion:

The source of the problem is (mostly I think) that when dns_get_record() cannot find a name, it returns false (good thing), but also emits a PHP warning.

Find a way to fix this up, or trap and handle the warning or...

This will help with by reducing the "dud" warnings that are nothing to do with coding problems.

Associated revisions

Revision 642c6023 (diff)
Added by Phillip Davis over 3 years ago

Fix #6187 Handle Growl IP Address problems

Revision 0e9cb2ab (diff)
Added by Phillip Davis over 3 years ago

Fix #6187 Handle Growl IP Address problems

(cherry picked from commit 642c6023fea2957bb646b1290371ead508f5cc67)


#2 Updated by Jim Thompson over 3 years ago

  • Assignee set to Steve Beaver

#3 Updated by Steve Beaver over 3 years ago

  • Status changed from New to Feedback
  • Assignee changed from Steve Beaver to Phillip Davis

PR merged. Thanks Phil.

#4 Updated by Phillip Davis over 3 years ago

  • % Done changed from 0 to 100

#5 Updated by Chris Buechler about 3 years ago

  • Status changed from Feedback to Resolved


Also available in: Atom PDF