When I disable the "Enable ClamAV integration" option on the configuration screen, I still get the error displayed in my Status Report "Unable to connect to ClamAV service." Should not show error when service is disabled (even though module is enabled).

If it is still desired to report this status, it should be marked as informational and not error.

CommentFileSizeAuthor
#2 3117539-2-status.patch550 byteshey_germano

Issue fork clamav-3117539

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

mpotter created an issue. See original summary.

hey_germano’s picture

hey_germano
she/her
Primary language English
Location Minneapolis, MN, USA
commented
Status: Active » Needs review
StatusFileSize
new 3117539-2-status.patch550 bytes

Here's a patch to show the status as Info instead of an Error when the service is not enabled.

hey_germano’s picture

hey_germano
she/her
Primary language English
Location Minneapolis, MN, USA
commented
Assigned: Unassigned » hey_germano
Issue tags: +Global2020
d.steindl’s picture

d.steindl commented

The patch in #2 solves the issue.
Tested with ClamAV 2.0.0 in Drupal 9.2.10.

thirstysix’s picture

thirstysix commented
Status: Needs review » Reviewed & tested by the community

+ RTBC.
Tested with D9.3.3 with version 2.0.0

thirstysix’s picture

thirstysix commented
Version: 8.x-1.x-dev » 2.x-dev

Version 8.x-1.x-dev is Outdated.
Moving in to 2.x

hswong3i made their first commit to this issue’s fork.

hswong3i opened merge request !3

voleger’s picture

voleger
he/his
Primary language Ukrainian
Location Ukraine, Rivne
commented

+1 for RTBC

  • mcdruid committed e49bc85 on 2.x 
    Issue #3117539 by hey_germano, ThirstySix, mpotter, d.steindl: Do not...
mcdruid’s picture

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

Thanks!

Same comment as #3257034: Add missing clamav.permissions.yml file about creating MRs in issues which already have patches, especially without commenting about whether the MR has anything different in it.

Status: Fixed » Closed (fixed)

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