Problem/Motivation

Warning: Undefined variable $extensions_to_check in nagios_check_requirements() (line 461 of /is/htdocs/webdocs/web/modules/contrib/nagios/nagios.module)

TypeError: Drupal\nagios\Controller\RequirementsController::collectRequirements(): Argument #1 ($enabled_modules) must be of type array, null given, called in /is/htdocs/webdocs/web/modules/contrib/nagios/nagios.module on line 461 in Drupal\nagios\Controller\RequirementsController->collectRequirements() (line 41 of /is/htdocs/webdocs/web/modules/contrib/nagios/src/Controller/RequirementsController.php).

CommentFileSizeAuthor
#6 nagios-3404405-m6.patch1.01 KBsolideogloria

Issue fork nagios-3404405

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

sleitner created an issue. See original summary.

sleitner’s picture

sleitner commented
Priority: Normal » Major
Issue summary: View changes

sleitner opened merge request !6

sleitner’s picture

sleitner commented
Status: Active » Needs review

Please review.

solideogloria’s picture

solideogloria commented
Version: 8.x-1.18 » 8.x-1.x-dev
Priority: Major » Critical

Marking Critical, since Nagios isn't working at all right now due to this.

solideogloria’s picture

solideogloria commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new nagios-3404405-m6.patch1.01 KB

This is a patch of the MR, for ease of use with composer patches.

The fix worked for me.

sleitner’s picture

sleitner commented

@Gogowitsch, @greg.harvey, @hoanns, @kbahey, @McChicken, @instanceofjamie please release an new tagged version to restore functioniality.

solideogloria’s picture

solideogloria commented

Also consider marking 8.x-1.18 as "do not use" or "broken".

gogowitsch’s picture

gogowitsch commented

Thanks for your work, @solideogloria and @sleitner! I’ll release a new version later today. I am currently testing.

gogowitsch’s picture

gogowitsch commented
Status: Reviewed & tested by the community » Fixed

I have released 1.19 and my production server is green again.

I am deeply sorry for the time you have lost due to my poor quality assurance of the previous release! I have marked it as broken.

solideogloria’s picture

solideogloria commented

No problem. Thanks.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

gogowitsch closed merge request !6