Is syntax for db defaults for FilesystemScanExclude right?

[markVnl]

after installing (fresh) nethserver-clamscan-1.0.4-1.ns7.sdl.noarch

config show clamscan shows this for FilesystemScanExclude

    FilesystemScanExclude=/boot
/proc
/sys
/usr/share/doc

I wonder if this should not be : FilesystemScanExclude=/boot,/proc,/sys,/usr/share/doc

If so the syntax should be changed here:
https://github.com/stephdl/nethserver-clamscan/blob/ns7/root/etc/e-smith/db/configuration/defaults/clamscan/FilesystemScanExclude

also see:
https://community.nethserver.org/t/clamav-warning-cant-open-file-permission-denied/18836/27

[markVnl]

After further investigation, the browser cache has also to do with it.
If you update while having the cockpit clamscan open the validator fails.

Still think the above is valid.

Also IMVHO the 'sed' in the migration script is risky ...
maybe split on \n (newline) and join again with , is safer (just a thought)

[stephdl]

You are right, I changed the default property for FilesystemScanExclude

I did not understand your second hints, could you explain

[markVnl]

the sed is just a longshot because of the corrupted db Marco had...

I also installed version 1.0.3 and did update to 1.0.4 :
the validation of Exuded files failed..
Closing Firefox and login again it worked..

[stephdl]

Closing Firefox and login again it worked..

Yes this is valid for every module, you need to reload the page after an upgrade of a module, we know it. In fact cockpit restart itself after a while of if you disconnect the user.

Nothing I can do, after an upgrade do ctrl+F5 to reload the browser cache

[stephdl]

@markVnl does this PR could close this issue ?

#21

[markVnl]

Sure!
(Was only trying to be of some sort of help :) )

[stephdl]

much appreciated @markVnl because you highlighted a bug, the old default property still got \n separator.

thank again