Clamav socket

[teoberi]

According to the configuration file clamav/etc/clamd.conf.sample the local socket file is clamd.socket but in the configuration file squid-ecap-av/src/ecap_adapter_av.conf it appears avdsocket = /tmp/clamd.sock
To avoid changing the configuration file, it would be necessary to update avdsocket = /tmp/clamd.sock -> avdsocket = /tmp/clamd.socket
Are there any issues preventing this update?

[sp-andwei]

Seems reasonable.

[teoberi]

Sorry for that!
It seems that the latest version of Clamav 1.2.0 remodifies the socket name and location for (clamd.socket -> clamd.sock) to reflect the recommended in Docker images.
https://github.com/Cisco-Talos/clamav/releases/tag/clamav-1.2.0

Other improvements
...
The suggested path for the clamd.pid and clamd.sock file in the sample
configs have been updated to reflect the recommended locations for these files
in the Docker images. These are:

/run/clamav/clamd.pid
/run/clamav/clamd.sock
For consistency, it now specifies clamd.sock instead of clamd.socket

Patch courtesy of computersalat.

GitHub pull request: Cisco-Talos/clamav#931