Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

NGFW-14933: Phish_blocker is not detecting phishing emails #115

Merged
merged 1 commit into from
Jan 19, 2025

Conversation

singhrohit23
Copy link
Contributor

@singhrohit23 singhrohit23 commented Jan 15, 2025

This changes are required to support socket on 3310 port which is down with upgrade to 1.0.7

This ticket will cover daemon related changes. As with upgrade a new socket service is added to clamav, along with updater and daemon service . Earlier daemon was managing the port 3301 on a socket to process the information send by UVM, now new socket service will take care of managing it.

Reference
https://bbs.archlinux.org/viewtopic.php?id=233951
https://www.danami.com/clients/knowledgebase/195/ClamAV-will-not-start.-How-can-I-fix-the-ClamAV-error-daily.cvldinc-was-not-met.html?language=english

Changes required

  1. Remove following reference from clamd.conf , this will manage be managed by socket service
    TCPAddr 127.0.0.1
    TCPSocket 3310
    LocalSocket /var/run/clamav/clamd.ctl

Introduce a new override file to be managed by socket service
/etc/systemd/system/clamav-daemon.socket.d/override.conf[Socket]
ListenStream=
ListenStream=/run/clamav/clamd.ctl
ListenStream=127.0.0.1:3310

Daemon service is dependant on daily.cvd and main.cvd files , which is not present on fresh install and hence fails,
Here we are running command to (daily.cvd and main.cv) during installation of clamav. This will handle scenario even if freshclam service is running.

Verification 3310 port running
netstat -anp | grep -E "(Active|State|clam|3310)"

image

@singhrohit23 singhrohit23 force-pushed the ngfw-14933 branch 3 times, most recently from 927e664 to 539ec9e Compare January 16, 2025 09:41
@singhrohit23 singhrohit23 merged commit 87db53f into ngfw-release-17.3 Jan 19, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants