Bug#814772: proftpd-basic: includes ~-files in config, incluing a whole directory

[Hilmar Preuße]

On 14.12.2016 15:16, Francesco Paolo Lovergine wrote:

Hi Francesco,

> I would close this bug mainly as an old configuration issue.
> 
> At the time I added Include /etc/proftpd/conf.d 
> as a way to add a whole set of configuration files. The ~ for backup file is a
> convention (e.g. Emacs), so basically any plain file will be included, so the admin should
> ensure to include only valid and not redundant configuration files.
> 
> That said, _now_ I would use something like:
> 
> Include /etc/proftpd/conf.d/*.conf
> 
> which would obsolete this bug. This is _now_ a valid and supported include
> clause (not at the time of my first change in the configuration file).
> Changing that could add now more headaches for the poor maintainer, with
> people complaining about supposed broken configurations and so on.
> 
Meanwhile I agree w/ you that this is not a bug, I'd simply leave a
comment in the proftpd.conf that users should care about backup files
*~, *.bak etc.
Changing the default clause to /etc/proftpd/conf.d/*.conf seems a little
bit invasive to me.

Hilmar
-- 
#206401 http://counter.li.org