There should be a a notification/warning to show that there is not any security module installed to your website.

CommentFileSizeAuthor
#7 2977446-7.patch2.19 KBravi.kiran
#4 2977446-4.patch2.08 KBgchauhan
#2 2977446.patch1.99 KBgchauhan

Comments

gchauhan created an issue. See original summary.

gchauhan’s picture

Status: Active » Needs review
StatusFileSize
new1.99 KB

It will give a notification if there is not any security module enabled on your website.

anmolgoyal74’s picture

Status: Needs review » Needs work

Why there's need to call securitycheck_review_site() again.?

gchauhan’s picture

StatusFileSize
new2.08 KB
gchauhan’s picture

Status: Needs work » Needs review
anmolgoyal74’s picture

Status: Needs review » Needs work
Issue tags: +Needs reroll

Needs reroll.

ravi.kiran’s picture

StatusFileSize
new2.19 KB

Re-rolled patch in #4 against 8.x-1.x
Below is the merge conflict resolved during re-roll in file securitycheck.drush.inc

<<<<<<< HEAD
    $projects = update_get_available(true);
    foreach ($projects as $name => $project) {
      if (!isset($project["releases"])) {
        drush_set_error($name . " - No releases found ");
      }
      else {
        $info = system_get_info('module', $name);
        $version = $info['version'];
        $release = $project["releases"][$version];
        if (!empty($release["security"]) && strpos($release["security"], 'not') !== FALSE) {
          drush_set_error( $name . ' - ' . $release["security"]) ;
        }
      }
    };
    return drush_set_error(dt('Site requires further investigation.'));
=======
>>>>>>> Applying patch from issue 2977446 comment 12883742
ravi.kiran’s picture

Status: Needs work » Needs review