search

inverse-inc / packetfence

PacketFence is a fully supported, trusted, Free and Open Source network access control (NAC) solution. Boasting an impressive feature set including a captive-portal for registration and remediation, centralized wired and wireless management, powerful BYOD management options, 802.1X support, layer-2 isolation of problematic devices; PacketFence can be used to effectively secure networks small to very large heterogeneous networks.
https://packetfence.org
GNU General Public License v2.0
1.39k stars 291 forks source link

Chown warning with fingerbank-collector post-install #8351

Closed satkunas closed 2 weeks ago

satkunas commented 1 month ago

Debian 12 bookworm

Use : user-group separator with chown

https://github.com/inverse-inc/packetfence/blob/a4e19c976c558114ad1609eb33b0fdb622be3a7c/addons/perl-client/debian/fingerbank.postinst#L33

image

E-ThanG commented 1 month ago

Unrelated to the fingerbank warnings, but there are a few other times that chown with period is used.

addons/perl-client/Makefile:    chown fingerbank.fingerbank /usr/local/fingerbank/db/fingerbank_Local.db; \
addons/perl-client/Makefile:    chown fingerbank.fingerbank /usr/local/fingerbank/db/fingerbank_Upstream.db; \
addons/monit/monitoring-scripts/update.sh:execute_and_check "chown root.pf-monitoring $script_dir"
addons/monit/monitoring-scripts/update.sh:  execute_and_check "chown pf-monitoring.pf-monitoring $script_dir/$fname" "Cannot set executable bit on script"
addons/pfconfig/pfconfig.init:  chown -R $User.$User $control_files_dir
addons/AD/migrate.pl:pf_run("chown pf.pf $domain_config_file");
rpm/packetfence.spec:  chown pf.pf /usr/local/pf/conf/pf.conf
rpm/packetfence.spec:  chown pf.pf /usr/local/pf/conf/pfconfig.conf
bin/cluster/sync:            `chown pf.pf $file`;
JeGoi commented 1 month ago

Added. Thanks